UnsafeImplicitIntentLaunch lint error is not raised when running lint task.
Actual behavior
UnsafeImplicitIntentLaunch is reported 12 times.
Steps to reproduce the behavior
Remove UnsafeImplicitIntentLaunch from the lint baseline file and run lint task (after #21082 is merged).
From what I understood in my research, some Intent calls made for a system web browser are considered by lint to launch JetpackConnectionResultActivity, which only cares about wordpress://jetpack-connection?reason={error} type of links (not https). I'd appreciate a second check if this make sense. I'm also not sure what's the prefered solution - suppressing, or maybe there's something better like communicating to Intents that it should point to external components only?
dangermattic
commented
2 months ago
Expected behavior
UnsafeImplicitIntentLaunchlint error is not raised when running lint task.Actual behavior
UnsafeImplicitIntentLaunchis reported 12 times.Steps to reproduce the behavior
Remove
UnsafeImplicitIntentLaunchfrom the lint baseline file and run lint task (after #21082 is merged).From what I understood in my research, some Intent calls made for a system web browser are considered by lint to launch
JetpackConnectionResultActivity, which only cares aboutwordpress://jetpack-connection?reason={error}type of links (nothttps). I'd appreciate a second check if this make sense. I'm also not sure what's the prefered solution - suppressing, or maybe there's something better like communicating to Intents that it should point to external components only?