Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Webhook no longer updates repository information #798

Closed
JAORMX opened this issue Aug 30, 2023 · 0 comments · Fixed by #799
Closed

Webhook no longer updates repository information #798

JAORMX opened this issue Aug 30, 2023 · 0 comments · Fixed by #799
Assignees
@JAORMX

Comments

@JAORMX
Copy link
Contributor

JAORMX commented Aug 30, 2023

Toggling secret_scanning Did not result in changes to the policy status. This might be an issue either in the evaluation policy status update or we're getting cached results from GitHub.
@JAORMX JAORMX self-assigned this Aug 30, 2023
@JAORMX JAORMX mentioned this issue Aug 30, 2023
Merged
JAORMX added a commit that referenced this issue Aug 30, 2023
@JAORMX @jhrozek
Use IS NOT DISTINCT FROM for comparing rule eval status on upsert
efad2b2 
This fixes the upsert by using the `IS NOT DISTINCT FROM` postgres construct
when evaluating the conflict in the upsert call.

Closes: #798
Co-Authored-By: Jakub Hrozek <jakub@stacklok.com>
@jhrozek jhrozek mentioned this issue Aug 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JAORMX JAORMX

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use IS NOT DISTINCT FROM for comparing rule eval status on upsert mindersec/minder
1 participant
@JAORMX