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.

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

[Bug] Responder users fail to retrieve correct incidents under API #175

Open
giranm opened this issue Jul 27, 2022 · 2 comments
Open

[Bug] Responder users fail to retrieve correct incidents under API #175

giranm opened this issue Jul 27, 2022 · 2 comments
Labels
bug Something isn't working
Milestone

Comments

@giranm
Copy link
Owner

giranm commented Jul 27, 2022

Summary

This incident has been raised to identify and fix behaviours with calls to PagerDuty's API against two user types.

The implication of this bug means that responder users initially fetch the wrong incident dataset filled with duplicates, which are then removed on UI refresh from polling log_entries, furthermore showing another incorrect number of incidents for given query.

Current Behaviour

Global Admin

  • GIVEN a Global Admin base-role user
  • WHEN querying incidents that have multiple states
  • THEN the correct number of incidents is returned by PD API
  • Console debug - note unique incident numbers
    Screenshot 2022-07-27 at 15 34 40

Responder

  • GIVEN a Responder base-role user
  • WHEN querying incidents that have multiple states
  • THEN the incorrect number of incidents is returned by PD API
  • Console debug - note duplicate incident numbers
    Screenshot 2022-07-27 at 15 33 37

Expected Behaviour (Responder)

  • GIVEN a Responder base-role user
  • WHEN querying incidents that have multiple states
  • THEN the correct number of incidents is returned by PD API

Suggested Workaround

TESTED: Default to older pd.all("/incidents") method to fetch incidents using a linear pagination method

@giranm giranm added the bug Something isn't working label Jul 27, 2022
@giranm giranm added this to the Beta milestone Jul 27, 2022
@giranm giranm moved this to Triage in PagerDuty Live Jul 27, 2022
@giranm giranm changed the title [Bug] Responder users fail to retrieve correct incidents under pdParallelFetch [Bug] Responder users fail to retrieve correct incidents under API Jul 27, 2022
@giranm giranm moved this from Triage to In Progress in PagerDuty Live Jul 27, 2022
@giranm
Copy link
Owner Author

giranm commented Jul 27, 2022

Working with @martindstone on debugging this issue - it looks like PD Live's call to PD API returns duplicates for responder users.

However, when replicating this using native REST calls with Postman, we are seeing no duplicates in the responses.

@giranm
Copy link
Owner Author

giranm commented Aug 1, 2022

On further investigation, this looks to be an environmental issue with PagerDuty itself and not PD Live.
We have escalated this with PagerDuty engineering and will await further updates; unfortunately there is no workaround apart from granting users Global Admin base roles which is less than ideal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Blocked
Development

No branches or pull requests

1 participant