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

Summon. If not authorized (403), not exist (404), service error (500), do not call headless #230

Open

valentinedwv opened this issue Jul 19, 2023 · 1 comment

Assignees

Labels

Member

valentinedwv commented Jul 19, 2023

Hydroshare graciously returns an unauthorized, then gleaner goes out an trys again.
might need to go an add another field to runstats ;)

Screen Shot 2023-07-18 at 5 41 03 PM

valentinedwv added the bug label

valentinedwv assigned fils

Member Author

valentinedwv commented Jul 27, 2023

From Doug:
For the 403 / 500 stuff I think we can check at this point

gleaner/internal/summoner/acquire/acquire.go

Line 138 in 6a4d814

resp, err := client.Do(req)

and then just either skip or report. was just looking at the code during the meeting to see what it would take. Doesn't look onerous

valentinedwv changed the title ~~Summon. If not authorized, do not call headless~~ Summon. If not authorized (403), not exist (404), service error (500), do not call headless

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment