search

claroline / Claroline

Learning management system
https://www.claroline.com
Other
315 stars 188 forks source link

Send a message to all users of a workspace #474

Closed sleperf closed 10 years ago

sleperf commented 10 years ago

When we are on the list of user of a workspace, there is a feature to send a message to all user. But this command use a GET to redirect to the send message.

If we have a big nomber of user (1000 users for exemple), as the URL length is limited, we have a bad gateway error 502

A post should be used instead

Exemple with few user <>/message/show?ids[]=1&ids[]=3&ids[]=4&ids[]=6&ids[]=7&ids[]=8&ids[]=9&ids[]=10&ids[]=11&ids[]=12&ids[]=13&ids[]=14&ids[]=15&ids[]=16&ids[]=17&ids[]=18&ids[]=19&ids[]=20&ids[]=21&ids[]=22&ids[]=23&ids[]=25&ids[]=26&ids[]=27&ids[]=28&ids[]=29&ids[]=30&ids[]=31&ids[]=32&ids[]=33&ids[]=34&ids[]=35&ids[]=36&ids[]=37&ids[]=38&ids[]=39&ids[]=40&ids[]=41&ids[]=42&ids[]=43&ids[]=44&ids[]=45&ids[]=46&ids[]=48&ids[]=49&ids[]=50&ids[]=51&ids[]=52&ids[]=53&ids[]=54&ids[]=57&ids[]=58&ids[]=59&ids[]=61&ids[]=62&ids[]=63&ids[]=64&ids[]=65&ids[]=66&ids[]=67&ids[]=70&ids[]=71&ids[]=72&ids[]=73&ids[]=74&ids[]=75&ids[]=76&ids[]=77&ids[]=79&ids[]=80&ids[]=81&ids[]=82&ids[]=83&ids[]=84&ids[]=85

ngodfraind commented 10 years ago

Do you have the last version of the corebundle. It has been changed a while ago.

2014-04-09 16:54 GMT+02:00 LE PERF notifications@github.com:

When we are on the list of user of a workspace, there is a feature to send a message to all user. But this command use a GET to redirect to the send message.

If we have a big nomber of user (1000 users for exemple), as the URL length is limited, we have a bad gateway error 502

A post should be used instead

Exemple with few user

<>/message/show?ids[]=1&ids[]=3&ids[]=4&ids[]=6&ids[]=7&ids[]=8&ids[]=9&ids[]=10&ids[]=11&ids[]=12&ids[]=13&ids[]=14&ids[]=15&ids[]=16&ids[]=17&ids[]=18&ids[]=19&ids[]=20&ids[]=21&ids[]=22&ids[]=23&ids[]=25&ids[]=26&ids[]=27&ids[]=28&ids[]=29&ids[]=30&ids[]=31&ids[]=32&ids[]=33&ids[]=34&ids[]=35&ids[]=36&ids[]=37&ids[]=38&ids[]=39&ids[]=40&ids[]=41&ids[]=42&ids[]=43&ids[]=44&ids[]=45&ids[]=46&ids[]=48&ids[]=49&ids[]=50&ids[]=51&ids[]=52&ids[]=53&ids[]=54&ids[]=57&ids[]=58&ids[]=59&ids[]=61&ids[]=62&ids[]=63&ids[]=64&ids[]=65&ids[]=66&ids[]=67&ids[]=70&ids[]=71&ids[]=72&ids[]=73&ids[]=74&ids[]=75&ids[]=76&ids[]=77&ids[]=79&ids[]=80&ids[]=81&ids[]=82&ids[]=83&ids[]=84&ids[]=85

Reply to this email directly or view it on GitHubhttps://github.com/claroline/Claroline/issues/474 .

Godfraind Nicolas 5, rue de la Carrière 6592 Monceau-Imbrechies GSM: 0472 94 80 99

sleperf commented 10 years ago

Ok, we will then have the correction when we will update the core-bundle