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

[misc] user-agent tagging #997

Closed
5 tasks done
bkmartinjr opened this issue Feb 14, 2024 · 4 comments
Closed
5 tasks done

[misc] user-agent tagging #997

bkmartinjr opened this issue Feb 14, 2024 · 4 comments
Assignees
Labels
analytics Work relating to analytics support P0 Priority 0 - Critical, fix ASAP! python api Related to the API

Comments

@bkmartinjr
Copy link
Contributor

bkmartinjr commented Feb 14, 2024

On Python/R API-initiated HTTP requests, set User-Agent tags:

  • R/Py environment
  • Package version
  • Means to add additional "application name" tag (allows discrimination of CI, WMG builder, etc)

This should propagate through:

  • HTTP requests to CDN, eg, fetch of release.json, mirrors.json
  • TileDB requests to S3
  • Other requests, e.g., H5AD fetch via API which is currently using fsspec

To do:

For hints on setting the TileDB header config: TileDB-Inc/TileDB#4400
For common User-Agent header examples, see here

@bkmartinjr bkmartinjr added python api Related to the API r api labels Feb 14, 2024
@bkmartinjr bkmartinjr self-assigned this Feb 14, 2024
@pablo-gar pablo-gar added the P0 Priority 0 - Critical, fix ASAP! label Feb 21, 2024
@bkmartinjr bkmartinjr removed their assignment Mar 12, 2024
@metakuni metakuni added the analytics Work relating to analytics support label Apr 13, 2024
@ivirshup ivirshup mentioned this issue Jun 12, 2024
3 tasks
@ivirshup
Copy link
Collaborator

ivirshup commented Jul 3, 2024

We've merged #1193, which checks a few of the boxes here. I've edited the top comment to show this.

@pablo-gar
Copy link
Contributor

Other non-Census usage is now tracked here

chanzuckerberg/single-cell-data-portal#7298

@pablo-gar
Copy link
Contributor

R work is now tracked here
#1249

@pablo-gar pablo-gar removed the r api label Jul 22, 2024
@pablo-gar
Copy link
Contributor

pablo-gar commented Jul 22, 2024

@ivirshup if this has been completed

Builder fetch of datasets manifest needs to set header

Then we can close this ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analytics Work relating to analytics support P0 Priority 0 - Critical, fix ASAP! python api Related to the API
Projects
None yet
Development

No branches or pull requests

6 participants