Conversation
e18ab2b to
5f2b3a3
Compare
brunoocasali
left a comment
There was a problem hiding this comment.
I know it is not ready yet, but this is the correct header name! ;)
137aea6 to
96cbb42
Compare
96cbb42 to
eb54b77
Compare
eb54b77 to
72daf28
Compare
brunoocasali
left a comment
There was a problem hiding this comment.
@alallema the user agent implemented will be used to inform the scrapped pages that they are going to be scraped by Meilisearch.
But the one that we need is this one: meilisearch/integration-guides#150 which is the user-agent we send to meilisearch.
So I assume you had to pass it from Meilisearch Python.
044c385 to
fc89a6e
Compare
771: Adding a header parameters to the client constructor r=alallema a=alallema To enable and facilitate the integration of a user-agent into all SDKs. [see related issue](meilisearch/integration-guides#150) and specifically for [the docs-scraper integration](meilisearch/docs-scraper#387) a new parameter client_agent is now available in the client constructor. Co-authored-by: alallema <amelie@meilisearch.com> Co-authored-by: Amélie <alallema@users.noreply.github.com> Co-authored-by: Paul Sanders <psanders1@gmail.com>
86351c2 to
e5d627c
Compare
| ) | ||
|
|
||
| # Then | ||
| assert actual.meilisearch_client.http.headers['User-Agent'] == f"Meilisearch Python (v0.27.0);Meilisearch DocsScraper (v{__version__})" |
There was a problem hiding this comment.
The fact that all these headers are not separated by a white space makes matching harder on our analytics tools no?
It might not be a prefix search strategy though!
There was a problem hiding this comment.
That's a good question, I have no idea, but I think @brunoocasali will be better able to answer, as these entries will be used through Amplitude.
There was a problem hiding this comment.
Nope @bidoubiwa it is correct like this, they use the ; semicolon as the separator.
More about it here in this comment: meilisearch/integration-guides#150 (comment)
e5d627c to
b7b1931
Compare
Following this central issue meilisearch/integration-guides#150
This repo now add a user-agent in the header of docs-scraper