Skip to content

fix(aws-appsync): graphqlapi throws incorrect error message for authorizationConfig#15830

Merged
mergify[bot] merged 2 commits intoaws:masterfrom
peterwoodworth:fixErrorMessage
Jul 30, 2021
Merged

fix(aws-appsync): graphqlapi throws incorrect error message for authorizationConfig#15830
mergify[bot] merged 2 commits intoaws:masterfrom
peterwoodworth:fixErrorMessage

Conversation

@peterwoodworth
Copy link
Copy Markdown
Contributor

@peterwoodworth peterwoodworth commented Jul 29, 2021

fixes: #15039

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@gitpod-io
Copy link
Copy Markdown

gitpod-io bot commented Jul 29, 2021

@peterwoodworth peterwoodworth added @aws-cdk/aws-appsync Related to AWS AppSync pr-linter/exempt-test The PR linter will not require test changes effort/small Small work item – less than a day of effort labels Jul 29, 2021
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 30, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

aws-cdk-automation commented Jul 30, 2021

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: a21cade
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify mergify bot merged commit 1f23313 into aws:master Jul 30, 2021
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 30, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@peterwoodworth peterwoodworth added the contribution/core This is a PR that came from AWS. label Jul 30, 2021
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Aug 3, 2021
@peterwoodworth @TikiTDO
fix(aws-appsync): graphqlapi throws incorrect error message for autho…
4f48051 
…rizationConfig (aws#15830)

fixes: aws#15039

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
hollanddd pushed a commit to hollanddd/aws-cdk that referenced this pull request Aug 26, 2021
@peterwoodworth @hollanddd
fix(aws-appsync): graphqlapi throws incorrect error message for autho…
5583dd8 
…rizationConfig (aws#15830)

fixes: aws#15039

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@otaviomacedo otaviomacedo otaviomacedo approved these changes

Assignees

@otaviomacedo otaviomacedo

Labels

@aws-cdk/aws-appsync Related to AWS AppSync contribution/core This is a PR that came from AWS. effort/small Small work item – less than a day of effort pr-linter/exempt-test The PR linter will not require test changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

(aws-appsync): Bad error message - Missing default OIDC Configuration

4 participants

@peterwoodworth @aws-cdk-automation @otaviomacedo @woodwoop-ospo