search

OneSignal / OneSignal-iOS-SDK

OneSignal is a free push notification service for mobile apps. This plugin makes it easy to integrate your native iOS app with OneSignal. https://onesignal.com
Other
496 stars 263 forks source link

[Bug]: Is NOT allowed to set HSTS for main doc #1363

Closed David-Dixon closed 6 months ago

David-Dixon commented 9 months ago

What happened?

Users are not being registered with OneSignal, In the log I see various references like the below.

https://api.onesignal.com/apps//subscriptions/ is NOT allowed to set HSTS for main doc (null)

We are using the latest SDK 5.1.0

Steps to reproduce?

Install 5.1.0 SDK in your project.

What did you expect to happen?

Users to be registered.

OneSignal iOS SDK version

5.1.0

iOS version

15

Specific iOS version

iOS 17

Relevant log output

https://api.onesignal.com/apps/57778143-47ea-4514-b11a-35634973b494/subscriptions/3772680b-830c-4a68-b94b-588bdccdb7a8 is NOT allowed to set HSTS for main doc (null)

Code of Conduct

David-Dixon commented 9 months ago

Worth noting I have tried adding a App Transport Security exception.

David-Dixon commented 9 months ago

Resolved the users not being registered issue by changing the group ID.

group.com.mycompany.myapp

To

group.MYTEAMID.com.mycompany.myapp

Still get the print out in the log about HSTS but appears not to be causing any current issues.

emawby commented 9 months ago

Thank you for reporting. To confirm that it isn't causing issues are you able to update subscriptions (e.g. add tags to a user)?

emawby commented 6 months ago

Closing due to inactivity