Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Fix notification logic #2751

Merged
merged 3 commits into from Jan 19, 2023
Merged

Fix notification logic #2751

merged 3 commits into from Jan 19, 2023

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Jan 19, 2023

The existing logic was stopping the processing of a new file when no notification is configured on the container.
This prevented task from receiving new file notification

This bug was preventing the rest of the NewFils logic to proceed if no notification was associated with the container
@chkeita chkeita requested a review from tevoinea January 19, 2023 19:44
@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2023

Codecov Report

Merging #2751 (a7702aa) into main (61685e6) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #2751   +/-   ##
=======================================
  Coverage   29.08%   29.08%           
=======================================
  Files         321      321           
  Lines       38462    38462           
=======================================
  Hits        11188    11188           
  Misses      27274    27274           
Impacted Files Coverage Δ
...ce/ApiService/onefuzzlib/NotificationOperations.cs 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@chkeita chkeita enabled auto-merge (squash) January 19, 2023 21:58
@chkeita chkeita merged commit b1625d3 into microsoft:main Jan 19, 2023
@nharper285 nharper285 mentioned this pull request Jan 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants