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

Deleting a scheduled task in another tenant causes a 500 error instead of 401 #188

Open
ElanHasson opened this issue Jun 21, 2022 · 2 comments
Labels
bug Something isn't working good first issue Good for newcomers hacktoberfest Good issues for Hacktoberfest

Comments

@ElanHasson
Copy link
Member

Describe the bug

If you attempt to delete a scheduled task belonging to a different user, a 500 Error is returned.

image

Steps to reproduce

  1. Create scheduled task as User A
  2. Delete scheduled task as user B

Expected behaviour

A 401 Error should be returned.

@ElanHasson ElanHasson added the bug Something isn't working label Jun 21, 2022
@ElanHasson ElanHasson added good first issue Good for newcomers hacktoberfest Good issues for Hacktoberfest labels Aug 10, 2022
@joshuaauerbachwatson
Copy link

I am currently sometimes seeing 500 errors (instead of success) when deleting scheduled tasks. I am quite sure there is no change of user identity between creation and deletion. Possibly relevant: the interval between creation and deletion is shortish (a minute or two) and the scheduled task is enabled and firing every minute meanwhile. If the deletion request is then repeated, it succeeds.

I don't know whether this is related to this issue or not (strictly speaking it is not, based on the title of the issue) but I thought I'd report it here since it could be related.

@ElanHasson
Copy link
Member Author

Will look into this as part of this item as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers hacktoberfest Good issues for Hacktoberfest
Projects
None yet
Development

No branches or pull requests

2 participants