Add Metadata Directory of FlatFiles support#1292
Add Metadata Directory of FlatFiles support#1292pkolmann wants to merge 3 commits intosimplesamlphp:masterfrom
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... |
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
c06a17a to
a52c98d
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.