Skip to content

feat: Validate tiled ServiceAccount config at startup - #1548

Merged
tpoliaw merged 2 commits into
mainfrom
tiled-token-validation
Jun 24, 2026
Merged

tpoliaw merged 2 commits into
mainfrom
tiled-token-validation

Conversation

@tpoliaw

@tpoliaw tpoliaw commented May 28, 2026

Copy link
Copy Markdown
Contributor

Ensure that the tiled auth configuration is valid and fail fast instead of when the first data is written.

@tpoliaw
tpoliaw requested a review from a team as a code owner May 28, 2026 17:32
@codecov

codecov Bot commented May 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.86%. Comparing base (7fe3fda) to head (0482d81).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1548      +/-   ##
==========================================
+ Coverage   95.84%   95.86%   +0.02%     
==========================================
  Files          44       44              
  Lines        3276     3292      +16     
==========================================
+ Hits         3140     3156      +16     
  Misses        136      136              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tpoliaw
tpoliaw force-pushed the auth-checks branch 2 times, most recently from 5ed7d2e to 3b5b8a2 Compare May 29, 2026 10:27
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from ddf28f2 to 403846c Compare May 29, 2026 10:51
@tpoliaw
tpoliaw changed the base branch from auth-checks to opa-client May 29, 2026 10:51
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch 2 times, most recently from c23b4d7 to b575c2b Compare June 2, 2026 10:24
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from b575c2b to b6049c8 Compare June 3, 2026 10:49
Comment thread src/blueapi/config.py Resolved
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from b6049c8 to 63009c0 Compare June 5, 2026 14:27
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from 63009c0 to 9865265 Compare June 5, 2026 16:01
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from 9865265 to 41e28ca Compare June 8, 2026 20:05
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from 41e28ca to 1c0d35f Compare June 23, 2026 13:19
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from 1c0d35f to 494c3f6 Compare June 23, 2026 14:28
Base automatically changed from opa-client to main June 23, 2026 15:04
@tpoliaw
tpoliaw force-pushed the tiled-token-validation branch from 494c3f6 to 0482d81 Compare June 23, 2026 15:38
@tpoliaw
tpoliaw added this pull request to the merge queue Jun 24, 2026
Merged via the queue into main with commit dd3f40d Jun 24, 2026
19 checks passed
@tpoliaw
tpoliaw deleted the tiled-token-validation branch June 24, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants