-
Notifications
You must be signed in to change notification settings - Fork 678
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
Add Metadata Directory of FlatFiles support #1292
base: master
Are you sure you want to change the base?
Conversation
The MetaData Files usually are quite long and if you need to change something it's quite bulky to handle them. Diffing between different SPs is also not too easy. Thats why I added the support for Metadata config directories. Every SP goes into it's own flatfile.
Codecov Report
@@ Coverage Diff @@
## master #1292 +/- ##
============================================
+ Coverage 37.85% 38.00% +0.15%
- Complexity 3431 3446 +15
============================================
Files 129 130 +1
Lines 9738 9778 +40
============================================
+ Hits 3686 3716 +30
- Misses 6052 6062 +10 |
@tvdijen can you please advise if this change is worth accepting? |
Thanks @pkolmann!! I am |
@tvdijen could you take another look at this please. |
1c686ab
to
eb20457
Compare
08ebb9c
to
64fca25
Compare
7a53fc8
to
d73ae47
Compare
e5c0e21
to
d5616df
Compare
2e6ab04
to
32f9acc
Compare
7e3ea19
to
2523634
Compare
@tvdijen Any luck with merging this PR? This would very handy to have for larger installations... |
3b5f5ba
to
96357ee
Compare
7587851
to
d523b31
Compare
8c90121
to
d534e3b
Compare
bc1c5c8
to
d0a5974
Compare
ccb9b02
to
120a100
Compare
There is also an issue submitted wanting a similar functionality for the XML files for SPs. A comment over on that issue is calling for a more generic solution to allow both flatfile and xml to be loaded on demand: #1368 (comment) I mention this because the comment may not have been seen on this thread. I've been looking at this PR and the code a little bit in recent days. |
I had a bit of a play around with this and ended up with very similar code to what is here though trying to handle xml files as well if they are found in the metadata/subdirectory.d directory. I am not sure what I have in the below branch makes complete sense as I am very new to the SSP code base. It does seem that this PR could be extended to check for php files and do something on them and something else if an xml file was found, otherwise maybe just report an ignored file? |
Sorry about the wait. I have merged a PR with this functionality into the simplesamlphp-2.3 branch and master. The patch should also back port to a 2.2 release reasonably well if you would like to take it for a test run. |
6004a77
to
58bf8db
Compare
5c9fb2c
to
0970efc
Compare
c27831c
to
71e49f4
Compare
I like to split up long config files to split config files.
The MetaData Files usually are quite long and if you need to change
something it's quite bulky to handle them. Diffing between different SPs
is also not too easy.
Thats why I added the support for Metadata config directories. Every SP
goes into it's own flatfile.