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

Segmentation fault on Macbook M2 #1142

Closed
mehtaruchir opened this issue Oct 10, 2023 · 11 comments
Closed

Segmentation fault on Macbook M2 #1142

mehtaruchir opened this issue Oct 10, 2023 · 11 comments

Comments

@mehtaruchir
Copy link

mehtaruchir commented Oct 10, 2023

getting below error for saml2aws -a login command - specifically for macbook M2 , it works on intel x86 macbook

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x60 pc=0x1050428d8]

goroutine 1 [running]:
github.com/versent/saml2aws/v2/pkg/provider/browser.glob..func1({0x105720c98, 0x14000654000}, 0x1400012e4d0)
github.com/versent/saml2aws/v2/pkg/provider/browser/browser.go:98 +0x1b8
github.com/versent/saml2aws/v2/pkg/provider/browser.(*Client).Authenticate(0x140000b0840, 0x1400012e4d0)
github.com/versent/saml2aws/v2/pkg/provider/browser/browser.go:78 +0x214
github.com/versent/saml2aws/v2/cmd/saml2aws/commands.Login(0x14000020f40)
github.com/versent/saml2aws/v2/cmd/saml2aws/commands/login.go:105 +0x458
github.com/versent/saml2aws/v2/cmd/saml2aws/commands.loginRefreshCredentials(0x140001122a0?, 0x0?)
github.com/versent/saml2aws/v2/cmd/saml2aws/commands/console.go:105 +0x24
github.com/versent/saml2aws/v2/cmd/saml2aws/commands.loadOrLogin(0x14000022780, 0x0?, 0x140003fa060)
github.com/versent/saml2aws/v2/cmd/saml2aws/commands/console.go:83 +0x118
github.com/versent/saml2aws/v2/cmd/saml2aws/commands.Console(0x140003fa060)
github.com/versent/saml2aws/v2/cmd/saml2aws/commands/console.go:46 +0x80
main.main()
github.com/versent/saml2aws/v2/cmd/saml2aws/main.go:195 +0x6db4

@mehtaruchir
Copy link
Author

i found 1 behavior pattern,

now on mac M2 , saml2aws MFA only waits for 30 seconds , if one can not finish the MFA auth from their provider in 30 seconds then we get NIL pointer Segmentation exception.

@kbarlowgw
Copy link
Contributor

Sounds like the same issue I am having on the MacBook M1: #1139

@arobase-che
Copy link

arobase-che commented Oct 16, 2023

Same on Framework Laptop 13, doesn't seem to be relative to Apple.
I have to be quick to finish the 2FA auth.

@kbarlowgw
Copy link
Contributor

It seems to be playwright: https://playwright.dev/docs/api/class-page#page-wait-for-request

I was able to patch it locally: #1139 (comment)

@mehtaruchir
Copy link
Author

mehtaruchir commented Nov 2, 2023

I am still not able to patch it locally. I tried the steps given in #1139 , also the fix added in the #1153 is still not added in release, 2.36.12 version still has code till #1152 only . any plans for new version release ?

@mehtaruchir
Copy link
Author

kbarlowgw -- is it possible to create new release version. 2.36.13 ?

@kbarlowgw
Copy link
Contributor

kbarlowgw commented Nov 6, 2023 via email

@gliptak
Copy link
Contributor

gliptak commented Nov 14, 2023

is this fixed in current build? artifacts.zip at bottom of https://github.com/Versent/saml2aws/actions/runs/6855210834?pr=1162

@mehtaruchir
Copy link
Author

not able to test this . artifact.zip. unfortunately.

@gliptak
Copy link
Contributor

gliptak commented Nov 16, 2023

@mehtaruchir artifact.zip contains platforms specific executables enabling local testing

@tinaboyce
Copy link
Contributor

kbarlowgw -- is it possible to create new release version. 2.36.13 ?

@mehtaruchir there is currently a new release version. Are you able to see if the new version fixes the issue?

@mapkon mapkon closed this as completed May 1, 2024
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

No branches or pull requests

6 participants