Skip to content

[DOP-39630] Hide secrets from settings repr - #413

Merged
dolfinus merged 1 commit into
developfrom
improvement/DOP-39630
Aug 11, 2026
Merged

[DOP-39630] Hide secrets from settings repr#413
dolfinus merged 1 commit into
developfrom
improvement/DOP-39630

Conversation

@dolfinus

Copy link
Copy Markdown
Member

Change Summary

Discovered in MTSWebServices/horizon#227

Related issue number

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@dolfinus dolfinus self-assigned this Aug 11, 2026
@dolfinus
dolfinus force-pushed the improvement/DOP-39630 branch 5 times, most recently from f93e48f to e42408f Compare August 11, 2026 11:23
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
syncmaster/db/migrations
   env.py626214140%3–5, 7–13, 15–16, 23, 26–28, 31–32, 34, 37, 39, 45–47, 49–50, 52–59, 61–62, 64, 66, 69–73, 76, 88–89, 97–98, 101–102, 108–109, 112, 118, 124–125, 127, 130, 133, 136–137, 139
syncmaster/server
   __init__.py3530091%67–69
syncmaster/server/providers/auth
   dummy_provider.py61256457%54–56, 58–63, 65–68, 70–72, 79–80, 84, 89, 99–101, 111, 115
   keycloak_provider.py1031714383%66, 83–85, 119–121, 123->124, 124–125, 131->132, 132–133, 161->162, 162–163, 167–170
   oauth2_gateway_provider.py56146274%47->48, 48–50, 54–57, 67->68, 68–69, 79–80, 99, 112, 118
syncmaster/server/scripts
   manage_superusers.py8031201059%53->54, 54–56, 73->74, 74–76, 92–93, 95–97, 99–101, 103–104, 106, 110–111, 113, 115, 119–120, 122–125, 127–128, 130–131
syncmaster/server/settings/auth
   __init__.py1532176%42->43, 43–44, 49
syncmaster/settings
   broker.py1852170%12->13, 13–14, 52, 55–56
   database.py26114450%12–14, 16–19, 21, 74, 77–78
TOTAL540365383817787% 

@dolfinus
dolfinus force-pushed the improvement/DOP-39630 branch 4 times, most recently from 938691c to 6b0908c Compare August 11, 2026 11:50
@dolfinus
dolfinus force-pushed the improvement/DOP-39630 branch from 6b0908c to 50877aa Compare August 11, 2026 13:16
@dolfinus
dolfinus marked this pull request as ready for review August 11, 2026 13:17
@dolfinus
dolfinus merged commit 0b5ec95 into develop Aug 11, 2026
25 checks passed
@dolfinus
dolfinus deleted the improvement/DOP-39630 branch August 11, 2026 13:41
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