Skip to content

Conversation

@ErwinSteffens
Copy link
Contributor

@ErwinSteffens ErwinSteffens commented Oct 9, 2025

🔧 Changes

Fallback to access token from $HOME/.config/auth0/config.json for the given tenant.

When Keyring is not supported by the CLI (saving fails) the access token (without refresh token) is saved in the configuration file. Loading this token is not yet supported by Terraform provider.

📚 References

🔬 Testing

  • I have tested loading the access token from my config file on my Macbook.
  • Refactored the config loading to make testing easier.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 83.47107% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.78%. Comparing base (62b13c9) to head (5d83479).

Files with missing lines Patch % Lines
internal/config/config.go 83.47% 15 Missing and 5 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1381      +/-   ##
==========================================
+ Coverage   87.55%   87.78%   +0.22%     
==========================================
  Files         156      156              
  Lines       21888    21924      +36     
==========================================
+ Hits        19165    19245      +80     
+ Misses       2026     1973      -53     
- Partials      697      706       +9     
Files with missing lines Coverage Δ
internal/provider/provider.go 98.85% <ø> (ø)
internal/config/config.go 79.80% <83.47%> (+19.27%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@duedares-rvj
Copy link
Member

@ErwinSteffens Hello 👋
Thanks again for this contribution. I tried to rebase this with main and resolve the conflicts. It seems I dont have write access.
Could you either rebase or provide me write permissions?
We shall merge this then :)

@duedares-rvj
Copy link
Member

@ErwinSteffens 👋
Quick wave in case you missed the earlier comment :)

@ErwinSteffens
Copy link
Contributor Author

@duedares-rvj You should have access now.

@ErwinSteffens
Copy link
Contributor Author

@duedares-rvj I have update PR and fixed conflicts btw.

@duedares-rvj duedares-rvj merged commit f380671 into auth0:main Nov 13, 2025
3 checks passed
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.

4 participants