forked from spring-cloud-services-samples/acme-fitness-store
-
Notifications
You must be signed in to change notification settings - Fork 158
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
Connection String Modified for Order Service #184
Merged
dingmeng-xue
merged 5 commits into
Azure-Samples:Azure
from
yoshioterada:documents-modify
Nov 20, 2023
Merged
Connection String Modified for Order Service #184
dingmeng-xue
merged 5 commits into
Azure-Samples:Azure
from
yoshioterada:documents-modify
Nov 20, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If I push the link of the "Deploy to Azure" button, it failed to show the Form in the Azure Portal. And it seems that the link had changed as follows. Previous Link (Currently Failed) https://raw.githubusercontent.com/Azure-Samples/acme-fitness-store/Azure/workshops/azure-spring-apps-enterprise/full/03-workshop-environment-setup/acmedeploy.json) Now (Correct Link) https://raw.githubusercontent.com/Azure-Samples/acme-fitness-store/Azure/azure-spring-apps-enterprise/workshops/azure-spring-apps-enterprise/full/03-workshop-environment-setup/acmedeploy.json So I modified the link for it.
If I configured the Connection String for Npgsql (C#). Following error was showed and couldn't start the order service. So I modified to be able to access to the PostgreSQL from the catalog service. ```bash az keyvault secret set \ --vault-name acme-fitness-kv-yoshio \ --name "ConnectionStrings--OrderContext" \ --value "Server=$POSTGRES_SERVER.postgres.database.azure.com;Database=$ORDER_SERVICE_DB;Port=5432;SSL Mode=Require;Trust Server Certificate=true;User ID=$POSTGRES_SERVER_USER;Password=$POSTGRES_SERVER_PASSWORD" ``` ```text Unhandled exception. System.ArgumentException: To validate server certificates, please use VerifyFull or VerifyCA instead of Require. To disable validation, explicitly set 'Trust Server Certificate' to true. See https://www.npgsql.org/doc/release-notes/6.0.html for more details. ```
I modified the expression of the env variables
I modified the style with more visible. I also tried to the separate line and works fine.
dingmeng-xue
approved these changes
Nov 20, 2023
xtreme-brian-shim
pushed a commit
to svrc/acme-fitness-store
that referenced
this pull request
Aug 22, 2024
* Link missing for Deploy to Azure If I push the link of the "Deploy to Azure" button, it failed to show the Form in the Azure Portal. And it seems that the link had changed as follows. Previous Link (Currently Failed) https://raw.githubusercontent.com/Azure-Samples/acme-fitness-store/Azure/workshops/azure-spring-apps-enterprise/full/03-workshop-environment-setup/acmedeploy.json) Now (Correct Link) https://raw.githubusercontent.com/Azure-Samples/acme-fitness-store/Azure/azure-spring-apps-enterprise/workshops/azure-spring-apps-enterprise/full/03-workshop-environment-setup/acmedeploy.json So I modified the link for it. * Connection String Modified for Order Service If I configured the Connection String for Npgsql (C#). Following error was showed and couldn't start the order service. So I modified to be able to access to the PostgreSQL from the catalog service. ```bash az keyvault secret set \ --vault-name acme-fitness-kv-yoshio \ --name "ConnectionStrings--OrderContext" \ --value "Server=$POSTGRES_SERVER.postgres.database.azure.com;Database=$ORDER_SERVICE_DB;Port=5432;SSL Mode=Require;Trust Server Certificate=true;User ID=$POSTGRES_SERVER_USER;Password=$POSTGRES_SERVER_PASSWORD" ``` ```text Unhandled exception. System.ArgumentException: To validate server certificates, please use VerifyFull or VerifyCA instead of Require. To disable validation, explicitly set 'Trust Server Certificate' to true. See https://www.npgsql.org/doc/release-notes/6.0.html for more details. ``` * Modified Env Value I modified the expression of the env variables * I modified the style with more visible I modified the style with more visible. I also tried to the separate line and works fine.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If I configured the original Connection String for Npgsql (C#),
following error was showed and couldn't start the order service.
Following is the actual Error during the startup the service.
So I modified the Connection String as follows.
And the above command worked fine with following with following result.