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

Could you please make the nativeClientId optional in the headless CLI mode? #10039

Closed
2 tasks
m98 opened this issue Mar 22, 2022 · 5 comments
Closed
2 tasks
Labels
bug Something isn't working duplicate If marked with duplicate, issue will be closed & original will be added for traceability headless Headless mode for CI/CD p2

Comments

@m98
Copy link

m98 commented Mar 22, 2022

Is this feature request related to a new or existing Amplify category?

auth

Is this related to another service?

CLI

Describe the feature you'd like to request

I followed the Amplify Headless mode documentation to create a headless_pull.sh file.

In the Cognito user pool, I only have one app client which is for the website. There is no native app. But when I write the headless_pull.sh, it says: "auth headless is missing the following inputParams nativeClientId"
image

But for my case, I don't need any native app client, so I can't have a nativeClientId which is required.

Describe the solution you'd like

I think it would be great if you can make the native app client (nativeClientId) field optional.

Additional context

Even after adding a nativeClientId, I am getting a new error now which I saw it's already tagged as a bug, do you have any workaround for it?
"OAuth properties for the app clients are not matching."

image

Is this something that you'd be interested in working on?

  • 👋 I may be able to implement this feature request

Would this feature include a breaking change?

  • ⚠️ This feature might incur a breaking change
@josefaidt
Copy link
Contributor

Hey @m98 👋 thanks for raising this! This does appear to be a duplicate of the linked issue, #7821 , however as a workaround can you try specifying the same client ID for both app and native to see if that mitigates the issue for now?

@josefaidt josefaidt added bug Something isn't working duplicate If marked with duplicate, issue will be closed & original will be added for traceability p2 headless Headless mode for CI/CD labels Mar 22, 2022
@marcvberg marcvberg added p2 and removed p2 labels Mar 22, 2022
@m98
Copy link
Author

m98 commented Apr 12, 2022

I can confirm that using the same (duplicate) id for both web and native App solved the issue. But I hope the fix does not break it! Because I believe many people already have done this to temporarily fix the issue.

@agnelnieves
Copy link

Came into this issue because of the same reason.

@josefaidt
Copy link
Contributor

Closing in favor of tracking the existing issue, #7821

@josefaidt josefaidt closed this as not planned Won't fix, can't repro, duplicate, stale Aug 22, 2023
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate If marked with duplicate, issue will be closed & original will be added for traceability headless Headless mode for CI/CD p2
Projects
None yet
Development

No branches or pull requests

4 participants