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.

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

Warning if path is not below /home/groups #5

Closed
jochenkoschorke opened this issue Jan 15, 2015 · 1 comment
Closed

Warning if path is not below /home/groups #5

jochenkoschorke opened this issue Jan 15, 2015 · 1 comment
Assignees
Labels
Milestone

Comments

@jochenkoschorke
Copy link
Contributor

- group_config:

   - ac-testgroup-35: 
      - path: /var/ac-test

- ace_config:

   - ac-testgroup-35:
       - path: /content/geometrixx/en/ac-testing/area-01
         permission: allow
         privileges: jcr:all

leeds to
Installation triggered: Wed Apr 23 13:35:40 CEST 2014

2014-04-23 13:35:40.204: start merging configuration data from: /var/ac-tool/project-ac-tool-test-1/confix.txt
2014-04-23 13:35:40.205: start installation of merged configurations
2014-04-23 13:35:40.326: finished installation of groups configuration without errors!
2014-04-23 13:35:40.351: finished (transient) installation of access control configuration without errors!
2014-04-23 13:35:40.367: persisted changes of ACLs
2014-04-23 13:35:40.37: saved history in node: /var/statistics/achistory/history_1398252940368
2014-04-23 13:35:40.386: saved installed configuration files under : /var/statistics/achistory/history_1398252940368/installedConfigs

Execution time: 164 ms

Success: true

but the resulting path is /home/groups/var/ac-test
even worse if group is allready existing:

Installation triggered: Wed Apr 23 13:40:59 CEST 2014

2014-04-23 13:40:59.591: start merging configuration data from: /var/ac-tool/project-ac-tool-test-1/confix.txt
2014-04-23 13:40:59.593: start installation of merged configurations
2014-04-23 13:40:59.725: found change of intermediate path:
existing group: ac-testgroup-35 has intermediate path: /home/groups/var/ac-test
group from config: ac-testgroup-35 has intermediate path: /var/ac-test
recreated group with new intermediate path!
2014-04-23 13:40:59.735: finished installation of groups configuration without errors!
2014-04-23 13:40:59.756: finished (transient) installation of access control configuration without errors!
2014-04-23 13:40:59.772: persisted changes of ACLs
2014-04-23 13:40:59.775: saved history in node: /var/statistics/achistory/history_1398253259773
2014-04-23 13:40:59.792: saved installed configuration files under : /var/statistics/achistory/history_1398253259773/installedConfigs

Execution time: 182 ms

Success: true

Expected behaviour: An explicit Warnig should be rendered

@jochenkoschorke
Copy link
Contributor Author

Added missing home/groups path prefix to group path from bean.

@ugocei ugocei added this to the 1.2.6 milestone May 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants