Skip to content

feat: Maven plugin: support locally-stored PAT as fallback to TIMEFOLD_PAT env var - #2559

Draft
oemebamo wants to merge 1 commit into
TimefoldAI:mainfrom
oemebamo:timefold-solver-enterprise-issues-806
Draft

feat: Maven plugin: support locally-stored PAT as fallback to TIMEFOLD_PAT env var#2559
oemebamo wants to merge 1 commit into
TimefoldAI:mainfrom
oemebamo:timefold-solver-enterprise-issues-806

Conversation

@oemebamo

@oemebamo oemebamo commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

@sonarqubecloud

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

@mswiderski

Copy link
Copy Markdown
Contributor

I would prefer to avoid storing secrets (PAT in this case) in clear text in user's home directory. Since we are talking about maven plugin I would expect we follow standard practice of maven https://maven.apache.org/guides/mini/guide-encryption.html that way we could rely on exactly the same mechanism as all others plugins that require secrets to communicate with remote service. Plus this can also take advantage of encryption to not keep it clear text.

@oemebamo

oemebamo commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator Author

I would prefer to avoid storing secrets (PAT in this case) in clear text in user's home directory. Since we are talking about maven plugin I would expect we follow standard practice of maven https://maven.apache.org/guides/mini/guide-encryption.html that way we could rely on exactly the same mechanism as all others plugins that require secrets to communicate with remote service. Plus this can also take advantage of encryption to not keep it clear text.

excellent; thanks for the pointer - didn't know about this standard practice

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.

3 participants