question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Calling AuthorizationManagementActivity.createStartIntent causes the app to crash

See original GitHub issue

Checklist:

Configuration

  • Version: 0.8.1
  • Integration: native-Kotlin
  • Identity provider: our own

Issue Description

Calling AuthorizationManagementActivity.createStartIntent causes the app to crash because AuthorizationManagementRequest is package private.

Can I make a PR to change it to public?

Issue Analytics

  • State:closed
  • Created 2 years ago
  • Reactions:1
  • Comments:5

github_iconTop GitHub Comments

1reaction
agologancommented, Apr 26, 2021

While I do not agree with your statements, having reevaluated that API it was a mistake that it’s currently unusable although being public.

0reactions
Nimroddacommented, Apr 26, 2021

While I understand your intent this library is explicitly designed to not use an embedded WebView (as mentioned in the first paragraph of our Readme). Adding a change that may encourage that usage is not something that goes along with that philosophy even if it is for an exceptional use-case.

I understand your point and see how this can be a security issue, but I think this applies more when using a 3rd party provider, such as Google and Facebook, which isn’t the case here. In our case, we have our own provider, which is used only in our apps.

Alternatively, what I would encourage you here is to ask your users to install a browser they trust. https://tools.ietf.org/html/rfc8252#section-8.12 expands more on the topic.

We have this solution currently in place and the UX is so bad that we started getting negative reviews about it. Some users don’t even bother to continue with the sign in. We have apps with millions of users. Having users not being able to log in means a direct hit to our revenue.

IMO, the library shouldn’t enforce anything, but leave it to the developers. Everyone has a different use case and should be able to make the decision what’s best for their app.

Read more comments on GitHub >

github_iconTop Results From Across the Web

CentralHeatPumpSystem Control Schedule default value and ...
Calling AuthorizationManagementActivity.createStartIntent causes the app to crash, 5, 2021-04-23 ; Unable to pull docker image, 0, 2021-01-14 ; Taskbar is not ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found