Skip to content

Conversation

@aagbsn
Copy link
Contributor

@aagbsn aagbsn commented Nov 6, 2025

this adds a new environment variable "TOR_TARGETS" which points to a json file

this adds a new environment variable "TOR_TARGETS" which points to a json file
@aagbsn aagbsn force-pushed the add_tor_targets_to_ooniapi branch from c18384c to 77b980d Compare November 6, 2025 14:39
@aagbsn
Copy link
Contributor Author

aagbsn commented Nov 6, 2025

squashed fixes

failed_reports_bucket: str = (
"" # for uploading reports that couldn't be sent to fastpath
)
tor_targets: str = "" # path to json file containing Tor bridges and DirAuth endpoints
Copy link
Contributor

@LDiazN LDiazN Nov 7, 2025

Choose a reason for hiding this comment

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

We really can't have a file path here because of how this runs in AWS

This is deployed as an ECS task, so although it has a disk, the disk is recreated every time a new instance of the microservice is spawned. So we can't really assume the initial state of the disk

This cannot be part of an environment variable either since the file might be too long for parameter store (maximum length of standard parameter store is 4096 chars: https://docs.aws.amazon.com/systems-manager/latest/userguide/systems-manager-parameter-store.html#parameter-size-limits)

So maybe the only solution we have left is serving it from s3, but we still don't have a private bucket for this purpose

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

2 participants