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

Promrated needs authentication token and is failing silently #1738

Closed
LukeHackett12 opened this issue Jan 31, 2023 · 1 comment
Closed

Promrated needs authentication token and is failing silently #1738

LukeHackett12 opened this issue Jan 31, 2023 · 1 comment
Assignees
@LukeHackett12
Labels
protocol Protocol Team tickets

Comments

@LukeHackett12
Copy link
Contributor

🐞 Bug Report

Description

Promrated has stopped working as they made a breaking change to the endpoint we use to need auth. We didn't get notified as it fails silently.

Has this worked before in a previous version?

Yes, the previous version in which this bug was not present was: ....

🔬 Minimal Reproduction

run promrated

🔥 Error




31mERRO�[0m Getting validator statistics: not ok http response {"validator": "0xaa17636b71b5829e11e796df1dd24bf949b612139ce84eee9688f25d32b04c749b57d406f119c1eac21e848c2d16abb9", "body": "{\"detail\":\"Not authenticated\"}"}
@github-actions github-actions bot added the protocol Protocol Team tickets label Jan 31, 2023
@LukeHackett12 LukeHackett12 mentioned this issue Jan 31, 2023
Merged
@LukeHackett12 LukeHackett12 self-assigned this Jan 31, 2023
@LukeHackett12
Copy link
Contributor Author

Opened PR to panic so we will be alerted, still waiting for auth token off rated

obol-bulldozer bot pushed a commit that referenced this issue Jan 31, 2023
@LukeHackett12
testutil/promrated: add metric to count errors from rated api (#1739)
3360ba8 
Rated recently added authentication and this has been failing silently as the error was just being logged

category: bug
ticket: #1738
@LukeHackett12 LukeHackett12 mentioned this issue Jan 31, 2023
Merged
obol-bulldozer bot pushed a commit that referenced this issue Jan 31, 2023
@LukeHackett12
testutil/promrated: only report status code (#1746)
ec8f9be 
remove the endpoint from error metric

category: bug
ticket: #1738
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LukeHackett12 LukeHackett12

Labels
protocol Protocol Team tickets
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@LukeHackett12