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

Add tracking headers #266

Merged
merged 1 commit into from
Jul 5, 2024
Merged

Conversation

eccles
Copy link
Member

@eccles eccles commented Jun 4, 2024

Add DataTrails-User-Agent and DataTrails-Partner-ID headers.

Typical header values when called from datatrails-samples (for example will be:

{
"DataTrails-Partner-ID": "acmecorp",
"DataTrails-User-Agent": "datatrails-samples/v0.17.2 pysdk/v0.13.2"
}

test_results.txt

Fixes AB#9572

@eccles eccles requested review from JAG-UK and j-hartley June 4, 2024 15:03
@eccles eccles force-pushed the dev/paul/9572-add-user-agent-partner-id-headers branch 4 times, most recently from a2006c1 to d486fd4 Compare June 10, 2024 08:00
@eccles eccles marked this pull request as ready for review June 10, 2024 08:59
JAG-UK
JAG-UK previously requested changes Jun 24, 2024
archivist/archivistpublic.py Outdated Show resolved Hide resolved
examples/create_asset.py Show resolved Hide resolved
@eccles eccles requested a review from JAG-UK June 24, 2024 10:07
@eccles eccles force-pushed the dev/paul/9572-add-user-agent-partner-id-headers branch 4 times, most recently from 03d1228 to c66fb8f Compare July 3, 2024 14:19
Add Archivist-User-Agent and Archivist-Partner-ID headers.

Fixes AB#9572
@eccles eccles force-pushed the dev/paul/9572-add-user-agent-partner-id-headers branch from c66fb8f to 8172640 Compare July 3, 2024 16:37
@eccles eccles requested a review from rcoxhill-01 July 4, 2024 13:13
@eccles eccles merged commit 692b016 into main Jul 5, 2024
12 checks passed
@eccles eccles deleted the dev/paul/9572-add-user-agent-partner-id-headers branch July 5, 2024 15:54
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