Skip to content

Fix dat.ttl_seconds default value mismatch in documentation#11966

Open
Annu881 wants to merge 4 commits intocBioPortal:masterfrom
Annu881:fix/dat-ttl-seconds-docs-11921
Open

Fix dat.ttl_seconds default value mismatch in documentation#11966
Annu881 wants to merge 4 commits intocBioPortal:masterfrom
Annu881:fix/dat-ttl-seconds-docs-11921

Conversation

@Annu881
Copy link

@Annu881 Annu881 commented Feb 17, 2026

Fixes #11921

Describe changes proposed in this pull request.
Fixes documentation mismatch for dat.ttl_seconds: documented default was 2592000 (30 days) while actual implementation uses -1.

Updated the following documentation files to reflect the correct default value of -1:

  • docs/deployment/authorization-and-authentication/Authenticating-Users-via-Tokens.md

  • docs/deployment/customization/security.properties-Reference.md

  • src/main/resources/security.properties.EXAMPLE
    Code references showing actual default value:

      UidDataAccessTokenServiceImpl.java:59
     @Value("${dat.ttl_seconds:-1}")
    
    JwtUtils.java:80
    @Value("${dat.ttl_seconds:-1}")
    

Checks

Any screenshots or GIFs?

Not applicable - documentation-only change

Notify reviewers

@inodb @dippindots @mavwolverine

- Modified getDataAccessToken to check if token list is empty
- If empty, create a new token instead of throwing IndexOutOfBoundsException
- Added test case testGetDataAccessTokenWhenNoTokensExist to verify fix

Fixes cBioPortal#11905
- Update README.md with v7 release info
- Add v6 -> v7 migration section to Migration-Guide.md
- Update Release-Procedure.md with branching notes
- Update ClickHouse mode section in Docker deployment docs
@sonarqubecloud
Copy link

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.

[BUG] dat.ttl_seconds default value mismatch in docs and code

1 participant