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.

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

Safe for use from concurrent goroutines? #6

Closed
carbocation opened this issue Mar 9, 2016 · 4 comments
Closed

Safe for use from concurrent goroutines? #6

carbocation opened this issue Mar 9, 2016 · 4 comments

Comments

@carbocation
Copy link

Is the client safe for use from concurrent goroutines?

@sideshow
Copy link
Owner

Yes it it - It uses the underlying Transport which is safe for concurrent use by multiple goroutines. We are using it in production in this manner.

@carbocation
Copy link
Author

I think it's worth mentioning this in the comments (for example, database/sql does so). I can create a pull request if that would be worthwhile.

@sideshow
Copy link
Owner

Great idea, yes please!

@sagarp-webonise
Copy link

To avoid fatal error of concurrent map read and write, are you using Push or PushWithContext? I am using this package and concurrent go routines trying to Push() notification always results with fatal error. Can you help me with sample example to avoid this fatal error?

ringsaturn referenced this issue in ringsaturn/apns2 Apr 22, 2023
[CH-4323] Add Live Activity Support
sideshow pushed a commit that referenced this issue Aug 8, 2024
* Merge pull request #6 from braze-inc/CH-4323
* [CH-4323] Add Live Activity Support
* [CH-5411] Add support for Attributes/AttributesType for push-to-start live activities (#7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants