Skip to content

Propagate auth errors from service#3609

Open
MDrakos wants to merge 8 commits into
betafrom
DX-3164
Open

Propagate auth errors from service#3609
MDrakos wants to merge 8 commits into
betafrom
DX-3164

Conversation

@MDrakos

@MDrakos MDrakos commented Nov 26, 2024

Copy link
Copy Markdown
Member
StoryDX-3164 We have a message queue that informs users of auth issues

Sample:
image

@github-actions github-actions Bot changed the base branch from master to version/0-48-0-RC1 November 26, 2024 23:21
@mitchell-as

Copy link
Copy Markdown
Collaborator

It's not clear to me what other authentication issues could happen that ought to be captured by the poller. The ticket does not make it clear which cases we should expect to have happen and handle. The ACTIVESTATE_API_KEY env var Mike handled in this PR and which is used in this description seems to be the only one.

Putting this up for early review. Maybe it's enough for now.

@mitchell-as mitchell-as requested a review from Naatan February 27, 2025 22:52
@mitchell-as mitchell-as marked this pull request as ready for review February 27, 2025 22:52
@Naatan

Naatan commented Feb 28, 2025

Copy link
Copy Markdown
Contributor

@MDrakos I know it's been a while; do you recall if there was still work missing from this PR?

@Naatan

Naatan commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

@MDrakos ping

@MDrakos

MDrakos commented Mar 27, 2025

Copy link
Copy Markdown
Member Author

@Naatan Bah sorry not sure how I lost track of this. I don't think there was any functional work left to do just maybe some improvements do how the code is structured.

@Naatan Naatan removed their request for review June 17, 2025 17:17
Base automatically changed from version/0-48-0-RC1 to beta September 15, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants