{"action":"created","issue":{"url":"https://api.github.com/repos/cgeo/cgeo/issues/5965","repository_url":"https://api.github.com/repos/cgeo/cgeo","labels_url":"https://api.github.com/repos/cgeo/cgeo/issues/5965/labels{/name}","comments_url":"https://api.github.com/repos/cgeo/cgeo/issues/5965/comments","events_url":"https://api.github.com/repos/cgeo/cgeo/issues/5965/events","html_url":"https://github.com/cgeo/cgeo/issues/5965","id":177180365,"number":5965,"title":"java.lang.SecurityException accessing location through gms","user":{"login":"rsudev","id":780005,"avatar_url":"https://avatars.githubusercontent.com/u/780005?v=3","gravatar_id":"","url":"https://api.github.com/users/rsudev","html_url":"https://github.com/rsudev","followers_url":"https://api.github.com/users/rsudev/followers","following_url":"https://api.github.com/users/rsudev/following{/other_user}","gists_url":"https://api.github.com/users/rsudev/gists{/gist_id}","starred_url":"https://api.github.com/users/rsudev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rsudev/subscriptions","organizations_url":"https://api.github.com/users/rsudev/orgs","repos_url":"https://api.github.com/users/rsudev/repos","events_url":"https://api.github.com/users/rsudev/events{/privacy}","received_events_url":"https://api.github.com/users/rsudev/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cgeo/cgeo/labels/Bug","name":"Bug","color":"e10c02"},{"url":"https://api.github.com/repos/cgeo/cgeo/labels/Google%20Play%20related","name":"Google Play related","color":"DDDDDD"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-09-15T13:42:50Z","updated_at":"2016-09-15T16:37:34Z","closed_at":null,"body":"We regularly get such crashes fro Android 6.0 and 7.0 devices. I could reproduce it by denying location access for c:geo in the app settings.\r\nPerhaps we should try to expect this, capture it and inform the user accordingly.\r\n\r\n```\r\njava.lang.SecurityException: Client must have ACCESS_FINE_LOCATION permission to request PRIORITY_HIGH_ACCURACY locations.\r\n\tat android.os.Parcel.readException(Parcel.java:1620)\r\n\tat android.os.Parcel.readException(Parcel.java:1573)\r\n\tat com.google.android.gms.location.internal.zzi$zza$zza.zza(Unknown Source)\r\n\tat com.google.android.gms.location.internal.zzk.zza(Unknown Source)\r\n\tat com.google.android.gms.location.internal.zzl.zza(Unknown Source)\r\n\tat com.google.android.gms.location.internal.zzd$7.zza(Unknown Source)\r\n\tat com.google.android.gms.location.internal.zzd$7.zza(Unknown Source)\r\n\tat com.google.android.gms.internal.zzpr$zza.zzb(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqa.zzf(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqa.zzd(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqf.zzd(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqd.zzd(Unknown Source)\r\n\tat com.google.android.gms.location.internal.zzd.requestLocationUpdates(Unknown Source)\r\n\tat cgeo.geocaching.playservices.LocationProvider.updateRequest(LocationProvider.java:54)\r\n\tat cgeo.geocaching.playservices.LocationProvider.onConnected(LocationProvider.java:140)\r\n\tat com.google.android.gms.common.internal.zzl.zzo(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqd.zzm(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqb.zzapm(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqb.onConnected(Unknown Source)\r\n\tat com.google.android.gms.internal.zzqf.onConnected(Unknown Source)\r\n\tat com.google.android.gms.internal.zzpu.onConnected(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzk$1.onConnected(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzd$zzj.zzarz(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzd$zza.zzc(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzd$zza.zzx(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzd$zze.zzasb(Unknown Source)\r\n\tat com.google.android.gms.common.internal.zzd$zzd.handleMessage(Unknown Source)\r\n\tat android.os.Handler.dispatchMessage(Handler.java:102)\r\n\tat android.os.Looper.loop(Looper.java:148)\r\n\tat android.app.ActivityThread.main(ActivityThread.java:5461)\r\n\tat java.lang.reflect.Method.invoke(Native Method)\r\n\tat com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)\r\n\tat com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)\r\n```"},"comment":{"url":"https://api.github.com/repos/cgeo/cgeo/issues/comments/247381796","html_url":"https://github.com/cgeo/cgeo/issues/5965#issuecomment-247381796","issue_url":"https://api.github.com/repos/cgeo/cgeo/issues/5965","id":247381796,"user":{"login":"samueltardieu","id":44656,"avatar_url":"https://avatars.githubusercontent.com/u/44656?v=3","gravatar_id":"","url":"https://api.github.com/users/samueltardieu","html_url":"https://github.com/samueltardieu","followers_url":"https://api.github.com/users/samueltardieu/followers","following_url":"https://api.github.com/users/samueltardieu/following{/other_user}","gists_url":"https://api.github.com/users/samueltardieu/gists{/gist_id}","starred_url":"https://api.github.com/users/samueltardieu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samueltardieu/subscriptions","organizations_url":"https://api.github.com/users/samueltardieu/orgs","repos_url":"https://api.github.com/users/samueltardieu/repos","events_url":"https://api.github.com/users/samueltardieu/events{/privacy}","received_events_url":"https://api.github.com/users/samueltardieu/received_events","type":"User","site_admin":false},"created_at":"2016-09-15T16:37:34Z","updated_at":"2016-09-15T16:37:34Z","body":"And we should use the new priority model."}}
{"actor":{"display_login":"samueltardieu"}}