-
Notifications
You must be signed in to change notification settings - Fork 159
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
YAML support, Directory Support and compound syntax #71
Comments
Absolutely! Thanks so much for the amazing contribution, I'm so happy
I don't think it'll be too hard to adapt the code
Looking forward to the PR
…On Thu, Feb 6 2020 at 9:01 PM, Ivan Gregurić Ortolan < ***@***.*** > wrote:
Hi, as part of a project we implemented support for YAML format and
multi-file support. The YAML support allows to write a more human readable
permission config:
users: C=DE,O=org,OU=WEB,CN=t1.example.com,L=Stuttgart,ST=reg: topics:
Topic1: - Read TopicPref_*: - All groups: team1-app1-*: - Read - Describe
In the YAML example you can also see the compound syntax where you define All
to give all permissions on a topic.
The multi-file support allows to organize the permissions in different
files to make separation of concerns more easy.
The implementation was done around version 0.5 so I would need to invest
some time to create a valid pull request, are you interested in merging
this kind of features? Otherwise I will spare the time ^^'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
#71
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE7CW2MUVUMKUBUIYRCU4FLRBRUAHANCNFSM4KRDCPVQ
).
|
Ok then I will try to see first how much effort is to try merging back our fork and then decide if it makes more sense just to re-implement the changes. Can't promise on the timing though since I don't have that much currently, hopefully in the next two weeks.. |
Take your time :)
…On Thu, Feb 6 2020 at 9:13 PM, Ivan Gregurić Ortolan < ***@***.*** > wrote:
Ok then I will try to see first how much effort is to try merging back our
fork and then decide if it makes more sense just to re-implement the
changes. Can't promise on the timing though since I don't have that much
currently, hopefully in the next two weeks..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
#71
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE7CW2NM7NKBP2SMIJXETLLRBRVQDANCNFSM4KRDCPVQ
).
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, as part of a project we implemented support for YAML format and multi-file support. The YAML support allows to write a more human readable permission config:
In the YAML example you can also see the compound syntax where you define
All
to give all permissions on a topic.The multi-file support allows to organize the permissions in different files to make separation of concerns more easy.
The implementation was done around version 0.5 so I would need to invest some time to create a valid pull request, are you interested in merging this kind of features? Otherwise I will spare the time ^^'
The text was updated successfully, but these errors were encountered: