Skip to content

feat: add route_dependencies attribute to transaction extensions#885

Draft
hrodmn wants to merge 4 commits intomainfrom
feat/transactions-route-dependencies
Draft

feat: add route_dependencies attribute to transaction extensions#885
hrodmn wants to merge 4 commits intomainfrom
feat/transactions-route-dependencies

Conversation

@hrodmn
Copy link
Collaborator

@hrodmn hrodmn commented Feb 16, 2026

resolves #884

Related Issue(s):

Description:

This adds a new argument to the TransactionExtension and BulkTransactionExtension that makes it simple to add a dependency to all routes affected by the transactions extensions. It is possible that we would want to build off of the existing add_route_dependencies pathway instead of doing this within the extension, see #884 for some discussion around that point.

PR Checklist:

  • pre-commit hooks pass locally
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'STAC FastAPI Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.30.

Benchmark suite Current: a3ff0fd Previous: 2e2b0f9 Ratio
Items With Model validation (100) 116.6396950108403 iter/sec (stddev: 0.006995198306056394) 157.22439379368063 iter/sec (stddev: 0.00008602307183466387) 1.35

This comment was automatically generated by workflow using github-action-benchmark.

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.

add optional router dependencies for TransactionExtension and BulkTransactionsExtension to facilitate adding authentication to those endpoints

1 participant