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

Question on the Azure Search .Net SDK latency #7298

Closed
Ponant opened this issue Apr 17, 2018 · 3 comments
Closed

Question on the Azure Search .Net SDK latency #7298

Ponant opened this issue Apr 17, 2018 · 3 comments

Comments

@Ponant
Copy link

Ponant commented Apr 17, 2018

Hi,
It is probably a stupid question but I wonder if using Azure Search with the .Net SDK is not going to create an extra latency as we channel requests from the client to the web app controller which in turn will call the azure search engine, whereas the most straightforward will be to call the azure search instance directly from Javascript. Is that a correct reasoning?
Thanks

Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@BryanTrach-MSFT
Copy link
Member

@Ponant Thanks for the feedback! We are currently investigating and will update you shortly.

@brjohnstmsft
Copy link
Contributor

@Ponant Yes, there will be more latency if you have a web app between your client and your search service. However, we still recommend this architecture in most cases because otherwise you could be putting api-keys at risk by exposing them to the client. What we recommend is that you call your search service directly from JavaScript running in the browser only if you're fine with your data being publicly disclosed and your browser-based client uses a query api-key (definitely not an admin api-key).

@BryanTrach-MSFT
Copy link
Member

@brjohnstmsft Thank you for the reply.

@Ponant We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and tag me in your reply. We will gladly continue the discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants