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.

Sign up for GitHub

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

Jump to bottom

SumUp wants the microphone permission if bluetooth permission is not granted #84

Closed
mfhaberlach opened this issue Oct 23, 2019 · 4 comments
Closed

SumUp wants the microphone permission if bluetooth permission is not granted #84

mfhaberlach opened this issue Oct 23, 2019 · 4 comments

Comments

@mfhaberlach
Copy link

Hello, we're using the latest SumUp release 3.3 an noticed a strange behavior today.

If we revoke the bluetooth permission from our app and then log in to SumUp, a screen appears which tells me to grant the microphone permission.

When we grant the bluetooth permission and login to SumUp again, the screen did not appear.

That is pretty confusing for the users.

IMG_A91CC6670227-1
@mollidor
Copy link
Contributor

That is quite confusing indeed, thanks for raising. My hunch is that the account is configured to use our PIN+ reader which - depending on the hardware generation - is able to connect via an audio cable. So when the app lacks BT permissions it will fall back to audio and thus ask for Microphone access.

But I fully agree it is confusing and the screen is also horribly broken.

@mollidor
Copy link
Contributor

One more question: Is this a debug app you are running and is it lacking an icon file? We would have expected to see the app's icon in the last line - and yes, the string format is broken.

@mfhaberlach
Copy link
Author

Oh sorry, everything is fine with the AppIcon we just blacked it out, just the string format is wrong

mollidor added a commit that referenced this issue Nov 13, 2019
@mollidor
Version 3.4: Address an issue with trusted certificates on iOS 13
ab0ac58 
SDK versions v3.3 and earlier are currently not able to connect to
SumUp on iOS 13.3 Beta 1 and Beta 2.

* [UPDATE] Address an issue with changes to trusted certificates
  introduced with iOS 13, see https://support.apple.com/en-us/HT210176
* [BUGFIX] Fix an issue where the host app's bundle name would not be
  displayed properly. Reported as part of issue #84,
  see #84
@mollidor
Copy link
Contributor

Hi.

While not addressing the underlying audio permission issue v3.4 will at least fix the string formatting.

It also addresses important changed to TLS that come with iOS 13.3 so you will want to update to this version at your earliest convenience.

Regards,
Lukas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mollidor @mfhaberlach @jadeburton-sumup