simplesamlphp / simplesamlphp/simplesamlphp-module-aggregator2
Support for conditional GET in metadata aggregator (was #474)
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 5
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
From vis...@terena.org on February 08, 2012 23:23:03
Add support for conditional HTTP GET requests in the aggregator module. This would enable clients to efficiently poll for aggregated metadata.
Could be implemented using Last-Modified/If-Modified-Since and ETag/If-None-Match headers.
The aggregator2 module seems to already have some caching logic onboard, so it makes sense to implement it there.
Original issue: http://code.google.com/p/simplesamlphp/issues/detail?id=474
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the aggregator2 module's existing caching logic and the code that serves aggregated metadata over HTTP. Trace how response headers are currently produced, then verify that conditional requests using Last-Modified/If-Modified-Since and ETag/If-None-Match can return the appropriate not-modified response. Done means clients can efficiently poll without receiving unchanged metadata.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100