You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Unable to query metadata that ends in .xml or a / #298
This appears to be similar to #193. We are upgrading from 1.1.0 to 2.1.x. Currently I am testing 2.1.2. I am finding that, although all our SP metadata fiels are in our metadata aggregate used for MDQ, when entities ending in .xml or a trailing slash do not return metadata. They just return "Not found".
I apologize if this is a duplicate request, but I do not see any proposed solutions. How can we get pyff to query all entityID's no matter their spelling?
Pipeline with specifics removed:
- when update:
- load:
- /$my_aggregate.xml
- break
- when request:
- select
- pipe:
- when accept application/samlmetadata+xml application/xml:
- xslt:
stylesheet: tidy.xsl
- first
- sign:
key: /$my_key.keycert: /$my_cert.crt
- emit application/xml
- break
- when accept application/json:
- discojson
- emit application/json:
- break
E.G. for entityID "http://alation.com/" In the logs I see 2025-06-02 16:24:47 INFO pyff.builtins selecting using args: ['http://alation.com'] indicating the trailing slash is removed.
Activity
CzNorbi commentedon Jun 13, 2025
I'm facing a similar issue. Here's what I'm observing:
Don't loose track of trailing slashes
theseal commentedon Jun 24, 2025
I think I solved the problem with trailing slashes in #302. Main maintainer @mikaelfrykholm will take a look at it when back from vacation.
The content-type issues can be solved by setting the
--content_negotiation_policyaccording to easy to find documentation:pyFF/src/pyff/api.py
Lines 218 to 230 in 62fe3e2
It might be time to switch the default as stated in the docs.
theseal commentedon Jun 24, 2025
There is a cache problem when switching content-type. Will try to solve that as-well in the PR above.
CzNorbi commentedon Jun 24, 2025
I have also created a PR to fix this issue. #301
theseal commentedon Jun 25, 2025
Ah didn't see your PR before I started my work @CzNorbi.
Don't loose track of trailing slashes
theseal commentedon Jun 1, 2026
Solved by #302