Skip to content

fix: upgrade stac-fastapi to v6.1.5#561

Merged
ividito merged 2 commits into
developfrom
fix/stac-fastapi-upgrade
Jan 9, 2026
Merged

fix: upgrade stac-fastapi to v6.1.5#561
ividito merged 2 commits into
developfrom
fix/stac-fastapi-upgrade

Conversation

@ividito
Copy link
Copy Markdown
Collaborator

@ividito ividito commented Jan 5, 2026

Issue

#497

What?

  • Update PG env var names
  • Add flag to create writer connection pool for transactions
  • Add a bunch of new tests for transactions, which also makes local OIDC work better

There were inconsistencies between how unit tests were running and how the actual service was behaving with the transactions changes. I had a really hard time getting meaningful errors out of the unit tests, so I converted them into integration tests that run against the docker compose services and make use of the existing auth integrations we have there. This was way more promising and has me feeling confident about the changes, but I'm still not sure what's wrong with the unit tests.

Testing?

  • Tests pass Unit tests fail 🤔, manual + integration tests pass
  • Manually tested transactions, since the new default behaviour is to enable transactions with a new stac-fastapi environment variable. veda-backend already has a variable to enable the extension, which duplicates this functionality. We can continue to use this approach without breaking existing instances, or we can migrate to the alternative stac-fastapi variable.

Comment thread stac_api/runtime/src/config.py
@ividito ividito force-pushed the fix/stac-fastapi-upgrade branch from 3287a99 to 7c3a813 Compare January 8, 2026 17:05
@ividito
Copy link
Copy Markdown
Collaborator Author

ividito commented Jan 8, 2026

Unit tests fixed (thanks @smohiudd ), integration test changes moved over to #563

@ividito ividito marked this pull request as ready for review January 8, 2026 17:13
@ividito ividito merged commit 7e4f10d into develop Jan 9, 2026
4 checks passed
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.

3 participants