Skip to content

Make technology, geo, rank, and category parameters optional#83

Merged
max-ostapenko merged 2 commits intomainfrom
sore-orca
Jan 23, 2026
Merged

Make technology, geo, rank, and category parameters optional#83
max-ostapenko merged 2 commits intomainfrom
sore-orca

Conversation

@max-ostapenko
Copy link
Contributor

@max-ostapenko max-ostapenko commented Jan 17, 2026

Closes HTTPArchive/httparchive.org#1182

API Parameter Handling Improvements:

  • Made technology, geo, and rank parameters optional (defaulting to 'ALL') for the /adoption, /cwv, /lighthouse, /page-weight, and /audits endpoints.

Documentation Updates:

  • Updated the README.md to indicate that technology, geo, rank, and category parameters are now optional and default to 'ALL' where applicable.

Test Suite Enhancements:

  • Updated and expanded API route tests to verify that endpoints handle missing or empty technology, geo, and rank parameters by defaulting to 'ALL' and returning valid results, instead of returning errors.
  • Improved the shell API test script to include filter verification logic, ensuring that API responses match the requested filters and that default values are correctly applied.

…th 'ALL' default and enhance API filter tests.

Signed-off-by: Max Ostapenko <[email protected]>
@max-ostapenko
Copy link
Contributor Author

The requests are keep coming, I'd like to merge this.
@tunetheweb any reservations?

Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry didn't realise you were waiting on me. Merge away!

@max-ostapenko max-ostapenko merged commit eeac924 into main Jan 23, 2026
2 checks passed
@max-ostapenko max-ostapenko deleted the sore-orca branch January 23, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tech Report: no tech defined in API requests from the website

2 participants