firebase / firebase/firebase-admin-java

FirebaseMessaging.sendEach has a limit of 500 messages

Aperta
#1,103 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
api: messaging type: feature request
Lingua principale
Java
Stelle
620
Fork
305
Merge medio
3h 23m
PR unite (30g)
1

Descrizione

### [READ] Step 1: Are you in the right place?

* For issues or feature requests related to __the code in this repository__
file a Github issue.
* If this is a __feature request__ make sure the issue title starts with "FR:".
* For general technical questions, post a question on [StackOverflow](http://stackoverflow.com/)
with the firebase tag.
* For general Firebase discussion, use the [firebase-talk](https://groups.google.com/forum/#!forum/firebase-talk)
google group.
* For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
[Firebase support channel](https://firebase.google.com/support/).

Read and understood.

### [REQUIRED] Step 2: Describe your environment

* Operating System version: N/A
* Firebase SDK version: 9.4.2
* Library version: _____
* Firebase Product: messaging (auth, database, storage, etc)

### [REQUIRED] Step 3: Describe the problem

sendEach has a limit of 500 messages, this limit is probably a leftover from sendAll which no longer works. It's not really logical to have this limit anymore.

https://github.com/firebase/firebase-admin-java/blob/master/src/main/java/com/google/firebase/messaging/FirebaseMessaging.java#L226

#### Steps to reproduce:

Create a list with over 500 messages / push notifications. Call FirebaseMessaging.sendEach with the list. An error is thrown that there is over 500 messages in the array.
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.

#### Relevant Code:

```
checkArgument(immutableMessages.size() <= 500,
"messages list must not contain more than 500 elements");
```
https://github.com/firebase/firebase-admin-java/blob/master/src/main/java/com/google/firebase/messaging/FirebaseMessaging.java#L226

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inspect src/main/java/com/google/firebase/messaging/FirebaseMessaging.java around line 226, starting with the validation applied by FirebaseMessaging.sendEach. Confirm whether the 500-message restriction still applies, then locate the relevant tests and update them so completion is demonstrated by handling a list larger than 500 without that validation error.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
api, backend
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.