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

Bug fix: Endpoint parameter is incorrect #118

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RohanKrishna-2000
Copy link

Documentation asks user to replace the endpoint parameter mentioned with the one specified in Azure portal("https://api.bing.microsoft.com" + attached image below) . This is incorrect as during runtime, an exception is thrown. "404 Client Error: Resource Not Found for url: https://api.bing.microsoft.com/?q=Microsoft+Bing+Search+Services&textDecorations=True&textFormat=HTML" Instead either one of the following endpoints should be used for the code to run as expected.
https://api.bing.microsoft.com/v7.0/search - For Web Search
https://api.bing.microsoft.com/v7.0/suggestions/search - For Autosuggest
https://api.bing.microsoft.com/v7.0/images/search - For Image Search
https://api.bing.microsoft.com/v7.0/videos/search - For Video Search
image

Documentation asks user to replace the endpoint parameter mentioned with the one specified in Azure portal("https://api.bing.microsoft.com") . This is incorrect as during runtime, an exception is thrown.
"404 Client Error: Resource Not Found for url: https://api.bing.microsoft.com/?q=Microsoft+Bing+Search+Services&textDecorations=True&textFormat=HTML"
Instead either one of the following endpoints should be used for the code to run as expected.
    https://api.bing.microsoft.com/v7.0/search	-  For Web Search
    https://api.bing.microsoft.com/v7.0/suggestions/search - For Autosuggest
    https://api.bing.microsoft.com/v7.0/images/search - For Image Search
    https://api.bing.microsoft.com/v7.0/videos/search - For Video Search
@learn-build-service-prod
Copy link

Learn Build status updates of commit 4d0e58d:

💡 Validation status: suggestions

File Status Preview URL Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md 💡Suggestion View Details

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

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.

1 participant