Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: enable egress tracking in production env #135

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

fforbeck
Copy link
Member

Enabled Egress Tracking in the production environment, along with the KV Delegation Store for saving and retrieving space/content/serve/* delegations.

@fforbeck fforbeck requested review from travis and alanshaw December 13, 2024 15:33
@fforbeck fforbeck self-assigned this Dec 13, 2024
GATEWAY_SERVICE_DID = "did:web:w3s.link"
UPLOAD_SERVICE_DID = "did:web:web3.storage"
CONTENT_CLAIMS_SERVICE_URL = "https://claims.web3.storage"
CARPARK_PUBLIC_BUCKET_URL = "https://carpark-prod-0.r2.w3s.link"
UPLOAD_API_URL = "https://up.web3.storage"
INDEXING_SERVICE_URL = "https://indexer.storacha.network/"
Copy link
Member Author

Choose a reason for hiding this comment

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

@alanshaw Could you confirm if this is the correct URL for the Production Indexer?

Copy link
Member

Choose a reason for hiding this comment

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

👍

Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

LGTM but please hold off pushing to prod until production indexing service is deployed.

@fforbeck
Copy link
Member Author

LGTM but please hold off pushing to prod until production indexing service is deployed.

sounds good. Thank you!

@fforbeck fforbeck force-pushed the chore/enable-egress-prod branch from 318bda9 to d9051b9 Compare December 16, 2024 14:20
@fforbeck fforbeck force-pushed the chore/enable-egress-prod branch from d9051b9 to 1fe97fe Compare December 17, 2024 16:53
@fforbeck fforbeck merged commit ad43b62 into main Dec 19, 2024
1 check passed
@fforbeck fforbeck deleted the chore/enable-egress-prod branch December 19, 2024 13:32
fforbeck pushed a commit that referenced this pull request Dec 19, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.22.0](v2.21.0...v2.22.0)
(2024-12-19)


### Features

* egress client - ucanto integration
([#123](#123))
([22bed68](22bed68))
* enable egress tracking in production env
([#135](#135))
([ad43b62](ad43b62))
* **indexer:** probabilistic feature flag
([#136](#136))
([dca2b35](dca2b35))
* ucan invocation handler
([#133](#133))
([b199bfa](b199bfa))
* Use Indexing Service when feature flag is present
([#132](#132))
([fa3f480](fa3f480))


### Bug Fixes

* **config:** staging kv bidding
([#134](#134))
([7a523d5](7a523d5))
* **egress-client:** set nonce, expire and fix servedAt
([#130](#130))
([b135643](b135643))
* enable open telemetry for all envs
([#131](#131))
([c822465](c822465))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
fforbeck added a commit to storacha/w3up that referenced this pull request Dec 19, 2024
### Context  
While updating the `web3-storage` docs, `w3cli`, and `@storacha/client`,
I found it inconvenient to repeatedly define the gateway DID and URL
every time I created a space. Also, if we don't provide the
`skipGatewayAuthorization=true`, it will throw an error saying it is
required to provide the gateway services to authorize.

### Changes
Following @travis suggestion (see:
storacha/docs#21 (comment)), I've
updated the client to automatically authorize the Storacha Gateway
(Production) by default if `skipGatewayAuthorization` is not set to
`true` and `authorizeGatewayServices` is not provided or is empty.

Once approved, this change will need to be ported to
[upload-service/w3up-client](https://github.com/storacha/upload-service/tree/main/packages/w3up-client).

And the PR storacha/freeway#135 needs to be
merged before this change gets released.
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