Skip to content

fix: global auth apis wire register for ea mode#6929

Merged
SATYAsasini merged 2 commits intomainfrom
oss-ea-global-config
Mar 3, 2026
Merged

fix: global auth apis wire register for ea mode#6929
SATYAsasini merged 2 commits intomainfrom
oss-ea-global-config

Conversation

@SATYAsasini
Copy link
Contributor

@SATYAsasini SATYAsasini commented Mar 3, 2026

Description

Fixes https://github.com/devtron-labs/sprint-tasks/issues/2839

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


Summary by Bito

  • Added import for globalConfigAPI package in router.go and wire.go
  • Added globalAuthorisationConfigRouter field to MuxRouter struct with improved alignment
  • Modified NewMuxRouter function to accept and initialize the new globalAuthorisationConfigRouter parameter
  • Added initialization of authorisationConfigRouter in the Init method
  • Included GlobalConfigWireSet in the wire dependency injection in wire.go

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@SATYAsasini SATYAsasini merged commit 634eb59 into main Mar 3, 2026
12 of 14 checks passed
@SATYAsasini SATYAsasini deleted the oss-ea-global-config branch March 3, 2026 12:52
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