-
Notifications
You must be signed in to change notification settings - Fork 482
Use new User Agent in CCDB #14884
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
Use new User Agent in CCDB #14884
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
Error while checking build/O2/fullCI_slc9 for ae35002 at 2025-12-09 16:40: Full log here. |
|
Is it possible to merge this despite the failing CI? I can not see how my change and the error can relate at all. (the error being the libc red black tree re-balance segmentation fault when ROOT Cling processes a macro file finding some ROOT plugin (maybe the zipfile plugin?)) |
|
I don't actually know what this is required for. So I would appreciate some more details in the PR message. Is it merely a simplification? What are the consequences for users? |
|
It's a common UserAgent string to be used by CCDB calls as well as JAliEn connections. Not important for users, but centrally we'll get a consistent information from both sides, as well as more information encoded in that string than we get today. So it's for us, not for the users. |
Apologies for being brief, this PR is about using the new HTTP User Agent string that I made available in libjalien02. An example would be // Old user agent string
b9p02p9794.cern.ch-1764695659-Jb0iA4-3470921716
// example new user agent string
ROOT-Notebook/python3@16946 target/Linux_x86_64 init/1764150428517 id/meide@meide-OptiPlex-7000 session/thaXU7tP MyDebugSymbol/MyDebugValue
There should not be any change in functionality from a users perspective. I guess one change for the users would be a different string in the logs. Also from a users perspective, the username of the user like aliprod, alihyperloop, or local name if testing locally, is also sent. |
|
Thanks for the explanation! I understand much better now. |
This changes the HTTP User Agent that is used when accessing CCDB.
The changes come from the libjalienO2 library.
For more details see this merge request.
Waiting for the new tag to be picked up alisw/alidist#6048It has now been approved