Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(agama): altering the AS configuration via config-api modifies rootDir of agama config #4523

Closed
jgomer2001 opened this issue Apr 6, 2023 · 0 comments · Fixed by #4524
Closed

fix(agama): altering the AS configuration via config-api modifies rootDir of agama config #4523

jgomer2001 opened this issue Apr 6, 2023 · 0 comments · Fixed by #4524
Assignees
@jgomer2001
Labels
comp-agama Touching folder /agama kind-bug Issue or PR is a bug in existing functionality

Comments

@jgomer2001
Copy link
Contributor

jgomer2001 commented Apr 6, 2023 •
edited
Loading

This is due to agama-model jar used in config api, where EngineConfig class has

private String rootDir = Path.of(System.getProperty("server.base"), "agama").toString();

which will produce a path with /jans-config-api instead of /jans-auth
@jgomer2001 jgomer2001 self-assigned this Apr 6, 2023
@mo-auto mo-auto added comp-agama Touching folder /agama kind-bug Issue or PR is a bug in existing functionality labels Apr 6, 2023
jgomer2001 added a commit that referenced this issue Apr 6, 2023
@jgomer2001
fix: about usage of java variable for Agama's root dir #4523
d10e2d3
@jgomer2001 jgomer2001 mentioned this issue Apr 6, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jgomer2001 jgomer2001

Labels
comp-agama Touching folder /agama kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: avoid setting agama configuration root dir based on java system variable JanssenProject/jans
2 participants
@jgomer2001 @mo-auto