framadate issueshttps://framagit.org/framasoft/framadate/framadate/-/issues2018-03-29T11:07:56Zhttps://framagit.org/framasoft/framadate/framadate/-/issues/292Fix ifneedbe checkbox2018-03-29T11:07:56ZThomasthomas.citharel@framasoft.orgFix ifneedbe checkboxhttps://screenshots.firefox.com/opvGhF3JrCI2SVCR/beta.framadate.org
https://screenshots.firefox.com/1SlKStwUr1xvGUEY/beta.framadate.orghttps://screenshots.firefox.com/opvGhF3JrCI2SVCR/beta.framadate.org
https://screenshots.firefox.com/1SlKStwUr1xvGUEY/beta.framadate.org1.1.0Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/293Fix error 414 - request URI too large when deleting long column name2021-09-30T10:50:32ZThomasthomas.citharel@framasoft.orgFix error 414 - request URI too large when deleting long column nameThis requires using the id of the choice and no longer it's value (why was it like that in the first place /o\ !)This requires using the id of the choice and no longer it's value (why was it like that in the first place /o\ !)1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/296Limit number of caracters for username in comments section2018-03-29T09:04:09ZThomasthomas.citharel@framasoft.orgLimit number of caracters for username in comments sectionThe database field is limited to 64 caracters, let's do the same with the fieldThe database field is limited to 64 caracters, let's do the same with the field1.1.0Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/302Laisser le commentaire rempli en cas d'erreur de nom2018-04-19T13:21:55ZMathieuLaisser le commentaire rempli en cas d'erreur de nomC'est dans le même ordre d'idée que le soucis suivant : https://framagit.org/framasoft/framadate/issues/194
Quand un commentaire est saisi avec un nom erroné, le commentaire s'efface. Comme il s'agit d'aide à la saisie, je pense qu'on p...C'est dans le même ordre d'idée que le soucis suivant : https://framagit.org/framasoft/framadate/issues/194
Quand un commentaire est saisi avec un nom erroné, le commentaire s'efface. Comme il s'agit d'aide à la saisie, je pense qu'on peut régler cela en JavaScript.1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/317Requête externe depuis l'éditeur markdown avancé2018-04-15T10:26:34ZRomain DessortRequête externe depuis l'éditeur markdown avancéL'éditeur markdown avancé pour la création des sondages (simplemde) fait une requête externe sur https://maxcdn.bootstrapcdn.com/font-awesome/latest/css/font-awesome.min.css pour Font Awesome.
[Il semble être possible](https://github.com...L'éditeur markdown avancé pour la création des sondages (simplemde) fait une requête externe sur https://maxcdn.bootstrapcdn.com/font-awesome/latest/css/font-awesome.min.css pour Font Awesome.
[Il semble être possible](https://github.com/sparksuite/simplemde-markdown-editor#configuration) de lui dire de ne pas le faire si le css est déjà présent localement.
Ça semble lié à cette issue #264 mais comme elle est assez générique j'ai ouvert celle-ci. N'hésite pas à la fermer si elle fait doublon.Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/318error on date poll creation2018-04-19T12:21:05ZMathieuerror on date poll creationquand on accède à l'URL `create_date_poll.php` en dehors des étapes "normales", un objet incorrect est mis dans `$_SESSION['form']` ce qui bloque la création de sondage.quand on accède à l'URL `create_date_poll.php` en dehors des étapes "normales", un objet incorrect est mis dans `$_SESSION['form']` ce qui bloque la création de sondage.1.2.0Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/322Better handle undefined config values2018-07-09T09:02:17ZMathieuBetter handle undefined config valuessuite à la fusion de la branche `fork-awesome` (!300), j'ai une erreur "undefined index" puisqu'il manque `provide_fork_awesome` dans mon fichier config.php.
est ce qu'on gère ce cas où les utilisateurs mettent à jour ?
ensuite, cette...suite à la fusion de la branche `fork-awesome` (!300), j'ai une erreur "undefined index" puisqu'il manque `provide_fork_awesome` dans mon fichier config.php.
est ce qu'on gère ce cas où les utilisateurs mettent à jour ?
ensuite, cette clé `provide_fork_awesome` a été ajouté dans `app/inc/config.template.php` mais est ce que ce fichier est encore utilisé ? de souvenir c'est le fichier `tpl/admin/config.tpl` qui est utilisé par le système d'installation.1.1.5https://framagit.org/framasoft/framadate/framadate/-/issues/330Undefined property: stdClass::$valuemax2019-09-09T15:08:16ZThomasthomas.citharel@framasoft.orgUndefined property: stdClass::$valuemaxAdd some coherence `valuemax` -> `valueMax`.Add some coherence `valuemax` -> `valueMax`.1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/331Don't push translations if json is invalid2018-05-28T08:04:29ZThomasthomas.citharel@framasoft.orgDon't push translations if json is invalidIf we have an invalid json file (say a comma leftover from a merge), json2po converts it to empty file and zanata pushes it, leading in all strings being destroyed on zanata :see\_no\_evil:
A security can be added by doing:
```bash
pyt...If we have an invalid json file (say a comma leftover from a merge), json2po converts it to empty file and zanata pushes it, leading in all strings being destroyed on zanata :see\_no\_evil:
A security can be added by doing:
```bash
python -m json.tool < locale/en.json &&
```
inside the `locales` section of the makefile before the json2po call.
@luc any better idea ?1.2.0Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/335Message trompeur dans l'étape 3 de création d'un sondage2019-07-02T10:17:10ZClermontéMessage trompeur dans l'étape 3 de création d'un sondageÀ la création d'un sondage, dans l'étape 3, des information sur l'archivage sont données :
> « Votre sondage sera automatiquement archivé 180 jours après le dernier jour de votre sondage. »
**Problème :** c'est faux. La date d'archiv...À la création d'un sondage, dans l'étape 3, des information sur l'archivage sont données :
> « Votre sondage sera automatiquement archivé 180 jours après le dernier jour de votre sondage. »
**Problème :** c'est faux. La date d'archivage/expiration du sondage est par défaut 180 jours après la date de création (la date du moment).
Fichier PollService.php :
```
/**
* @return int The max timestamp allowed for expiry date
*/
public function maxExpiryDate() {
global $config;
return time() + (86400 * $config['default_poll_duration']);
}
```
**Suggestions :**
- reformuler le message pour préciser ;
- ajouter la différence entre archiver, expiration et suppression ;
- donner une information sur la date de suppression (contrainte RGPD) ;
- faire pareil en anglais.1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/336Absence de contrôle de la date d'expiration en modification de sondage2022-04-20T10:59:03ZClermontéAbsence de contrôle de la date d'expiration en modification de sondageLors de la création du sondage, le code teste la date saisie par rapport à un minimum et un maximum. Dans create_classic_poll.php, ligne 72 :
```
if ($time < $min_expiry_time) {
$_SESSION['form']->end_date = $min_e...Lors de la création du sondage, le code teste la date saisie par rapport à un minimum et un maximum. Dans create_classic_poll.php, ligne 72 :
```
if ($time < $min_expiry_time) {
$_SESSION['form']->end_date = $min_expiry_time;
} elseif ($max_expiry_time < $time) {
$_SESSION['form']->end_date = $max_expiry_time;
} else {
$_SESSION['form']->end_date = $time;
}
```
Lors de la modification de la date d'expiration après que le sondage soit créé, il n'y a plus ce test. Dans app/classes/Framadate/Service/AdminPollService.php, ligne 37 :
```
function updatePoll($poll) {
global $config;
if ($poll->end_date > $poll->creation_date) {
return $this->pollRepository->update($poll);
} else {
return false;
}
}
```
**Problèmes :**
a) possibilité de mettre une date dans un futur lointain, très lointain… (voir ci-dessus : `if ($poll->end_date > $poll->creation_date) {`) ;
b) possibilité de mettre une date passée alors que factuellement des votes ont pu avoir lieu après cette date.
**Suggestions :**
1) modifier le code pour renforcer le test ;
2) mutualiser le code de test entre create_classic_poll.php, create_date_poll.php et AdminPollService.php.1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/343Make sure config.php has correct permissions2021-09-30T10:52:34ZThomasthomas.citharel@framasoft.orgMake sure config.php has correct permissionsSo that external attackers don't get to see database credentials.So that external attackers don't get to see database credentials.1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/344Day headers formatted wrong in certain browsers2018-09-12T20:39:09ZThomasthomas.citharel@framasoft.orgDay headers formatted wrong in certain browsers*This issue originally was at https://github.com/framasoft/framadate/issues/67*
I made a poll recently **on the public version of Framatade** of which the headers looked like this on my machine (**Arch Linux with Chrome 60.0.3112.40 (Of...*This issue originally was at https://github.com/framasoft/framadate/issues/67*
I made a poll recently **on the public version of Framatade** of which the headers looked like this on my machine (**Arch Linux with Chrome 60.0.3112.40 (Official Build) beta (64-bit)**):
![](https://i.imgur.com/cjMtOt4.png)
On my friend's computer (**Windows 7 Pro with Chrome 59.0.3071.115 (Official Build) (64-bit)**), they showed up like this:
![](https://i.imgur.com/4SXNFjK.png)
And on another friends mobile (no specs, can ask if requested), they showed up like this as well.
![](https://i.imgur.com/HpwfcDD.png)
My phone (**Android 6.0.1 with Chrome 59.0.3071.125**) it shows the normal column headers.
I guess it could also be a locale issue. I have all my locales set to US English, but considering the column header using "August", I guess the Windows is also configured in English.
--------
(The fact that on mobile it's impossible to see more than one or two columns (on my phone as well, even though the column headers show up correctly) is a separate issue. But if Framadate doesn't intend to be mobile-friendly, that's ok. Tell me if I should report that separately.)1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/345Comments not working on release 1.1.42018-07-09T13:56:24ZThomasthomas.citharel@framasoft.orgComments not working on release 1.1.4*This issue was originally at https://github.com/framasoft/framadate/issues/69*
Someone achieve to make comments working ?
When I click on submit nothing happens and no error in script..
The rest is ok ;)
Example url : https://coopeco-...*This issue was originally at https://github.com/framasoft/framadate/issues/69*
Someone achieve to make comments working ?
When I click on submit nothing happens and no error in script..
The rest is ok ;)
Example url : https://coopeco-supermarche.be/framadate/studs.php?poll=WvYNfXwuS6r4jEKm&user=Nicolas%20Jourdain&id=189
Have a nice dayhttps://framagit.org/framasoft/framadate/framadate/-/issues/352XSS security issue2018-07-09T09:18:24ZThomasthomas.citharel@framasoft.orgXSS security issueReported by https://www.bof.nl
> Dear Security Team,
> I noticed Post based XSS vulnerability on kies.bof.nl. There is Poll creation functionality on the website Which is consists of 3 steps. Vulnerability on the 2nd step which is poll ...Reported by https://www.bof.nl
> Dear Security Team,
> I noticed Post based XSS vulnerability on kies.bof.nl. There is Poll creation functionality on the website Which is consists of 3 steps. Vulnerability on the 2nd step which is poll date section.Time field is vulnerable.
>
> Vulnerable url:
> https://kies.bof.nl/create_date_poll.php?type=date
>
> Vulnerable parameter:
> horaires2[]
>
> Payload:
> " onmouseover="alert(document.domain)" bad="
>
> Replication steps:
> 1. go to https://kies.bof.nl/create_date_poll.php?type=date
> 2. field 1st step and press "Go to Step 2" button
> 3. on the 2nd step , choose any date from the calendar.
> 4. type above payload to the time 1, time 2, time 3 field.
> 5. Press "Next" Button
> 6. On the 3rt step, Press "Back" Button
> 7. Move mouse over the time field.
> 8. Popup will be prompted with "kies.bof.nl"
>
> Security Impact:
> Session Hijacking, Redirection to the Malicious website and many more.1.1.5Thomasthomas.citharel@framasoft.orgThomasthomas.citharel@framasoft.orghttps://framagit.org/framasoft/framadate/framadate/-/issues/357Framadate crashed on poll creation when PHPMailer send mail to localhost serv...2018-07-30T14:12:24ZChristian P. MOMONchristian.momon@devinsy.frFramadate crashed on poll creation when PHPMailer send mail to localhost server which is no TLS**Context**
* Framadate 1.1.7
* PHPMail 6.0.5
* local SMTP Server with NO TLS on Debian Stretch
* config.php :
* 'use_smtp' => true,
* 'smtp_options' => [
* 'host' => 'localhost',
* 'auth' => false,
* 'username' => '',
...**Context**
* Framadate 1.1.7
* PHPMail 6.0.5
* local SMTP Server with NO TLS on Debian Stretch
* config.php :
* 'use_smtp' => true,
* 'smtp_options' => [
* 'host' => 'localhost',
* 'auth' => false,
* 'username' => '',
* 'password' => '',
* 'secure' => false,
* 'port' => 25,
**Problem**
On last step of poll creation, Framadate crashed (empty page in browser) when sending mails.
Apache logs:
```
PHP Fatal error: Uncaught PHPMailer\\PHPMailer\\Exception: SMTP Error: Could not connect to SMTP host. in /var/www/date.chapril.org/vendor/phpmailer/phpmailer/src/PHPMailer.php:1900
Stack trace:
#0 /var/www/date.chapril.org/vendor/phpmailer/phpmailer/src/PHPMailer.php(1725): PHPMailer\\PHPMailer\\PHPMailer->smtpConnect(Array)
#1 /var/www/date.chapril.org/vendor/phpmailer/phpmailer/src/PHPMailer.php(1481): PHPMailer\\PHPMailer\\PHPMailer->smtpSend('Date: Sun, 29 J...', 'This is a multi...')
#2 /var/www/date.chapril.org/vendor/phpmailer/phpmailer/src/PHPMailer.php(1320): PHPMailer\\PHPMailer\\PHPMailer->postSend()
#3 /var/www/date.chapril.org/app/classes/Framadate/Services/MailService.php(58): PHPMailer\\PHPMailer\\PHPMailer->send()
#4 /var/www/date.chapril.org/create_date_poll.php(226): Framadate\\Services\\MailService->send('cmomon@april.or...', '[date.chapril][...', 'Ce message ne d...')
#5 {main}
th
```
Syslog:
```
Jul 29 15:30:07 lamp postfix/smtpd[32461]: connect from localhost[127.0.0.1]
Jul 29 15:30:07 lamp postfix/smtpd[32461]: lost connection after STARTTLS from localhost[127.0.0.1]
Jul 29 15:30:07 lamp postfix/smtpd[32461]: disconnect from localhost[127.0.0.1] ehlo=1 starttls=1 commands=2
```
The problem seems to come from PHPMailer configuration.
Watching PHPMailer.php file at line 270:
```
/**
* Whether to enable TLS encryption automatically if a server supports it,
* even if `SMTPSecure` is not set to 'tls'.
* Be aware that in PHP >= 5.6 this requires that the server's certificates are valid.
*
* @var bool
*/
public $SMTPAutoTLS = true;
```
Watching PHPMailer.php file at line 1898:
```
//Automatically enable TLS encryption if:
// * it's not disabled
// * we have openssl extension
// * we are not already using SSL
// * the server offers STARTTLS
if ($this->SMTPAutoTLS and $sslext and 'ssl' != $secure and $this->smtp->getServerExt('STARTTLS')) {
$tls = true;
}
if ($tls) {
if (!$this->smtp->startTLS()) {
throw new Exception($this->lang('connect_host'));
}
// We must resend EHLO after TLS negotiation
$this->smtp->hello($hello);
}
```
So, the PHPMailer comments say that TLS is used if available, and so we hope/wait that it will work without TLS if it is not available. But the source code throws an exception if the TLS try failed. There is no alternative: TLS or throw. Why force TLS?
Is-it a PHPMailer bug???
The consequence is: **Framadate + SMTP localhost server + NO TLS => FAILED**.
**Suggestions**
First, open an issue on the PHPMailer project.
Second, add an option parameter in config.php file to make possible to set the `$mail->SMTPAutoTLS = false` in MailService.php file.
Currently, I have to modify the Framadate MailService.php file to force SMTPAutoTLS to false:
```
private function configureMailer(PHPMailer $mailer) {
$mailer->isSMTP();
$mailer->SMTPAutoTLS = false; // <-- Temporary fix from Cpm 2018-07-29
```https://framagit.org/framasoft/framadate/framadate/-/issues/369Template syntax error when creating a poll2018-09-11T13:51:42ZMarc KarassevTemplate syntax error when creating a pollError:
`Fatal error: Uncaught --> Smarty Compiler: Syntax error in template "file:/var/www/framadate/tpl/create_poll_step_3.tpl" on line 36 "<p>{__('Step 3', 'Once you have confirmed the creation of your poll, you will automatically be ...Error:
`Fatal error: Uncaught --> Smarty Compiler: Syntax error in template "file:/var/www/framadate/tpl/create_poll_step_3.tpl" on line 36 "<p>{__('Step 3', 'Once you have confirmed the creation of your poll, you will automatically be redirected to the poll's administration page.')}</p>" - Unexpected "s", expected one of: "","" , ")" <-- thrown in /var/www/framadate/vendor/smarty/smarty/libs/sysplugins/smarty_internal_templatecompilerbase.php on line 36`
Steps to reproduce:
- checkout ´develop´ branch
- run `docker-compose up`
- go to `localhost`
- create whatever poll type
- go to poll creation step 3https://framagit.org/framasoft/framadate/framadate/-/issues/371Mobile version damaged on develop2018-11-05T15:25:53ZMarc KarassevMobile version damaged on developI tried to open in my mobile browser my running framadate instance but got the following output:![Screenshot_20180912-175326_Firefox](/uploads/be2591b7635a5bad12fae88c29ce3c3e/Screenshot_20180912-175326_Firefox.jpg)
whereas the output i...I tried to open in my mobile browser my running framadate instance but got the following output:![Screenshot_20180912-175326_Firefox](/uploads/be2591b7635a5bad12fae88c29ce3c3e/Screenshot_20180912-175326_Firefox.jpg)
whereas the output is normal on my computer's browser.
I'm using Firefox for Android 62.0.1 and got the same problem with Chrome for Android.
Steps to reproduce:
- clone the repository
- checkout the `develop` branch
- run `docker-compose up`
- access from your mobile browser to your running instance
Am I missing something?1.2.0https://framagit.org/framasoft/framadate/framadate/-/issues/384Lors d'ajout de colonne, les virgules sont transformées en tiret dans le titre2019-04-15T13:16:50ZThierry MunozLors d'ajout de colonne, les virgules sont transformées en tiret dans le titreBonjour,
J'ai remarqué que lorsque je rajoute une colonne à un sondage existant (sondage classique), le texte initial a les virgules transformées en tirets. Ceci n'arrive pas lors de la création.
Voici par exemple le titre que j'insère ...Bonjour,
J'ai remarqué que lorsque je rajoute une colonne à un sondage existant (sondage classique), le texte initial a les virgules transformées en tirets. Ceci n'arrive pas lors de la création.
Voici par exemple le titre que j'insère : `[Samedi 17/11/2018 : Sortie 8, Lagrasse-Ribaute, 1/2 journée](https://ornaisonskayak.com/event/sortie-8-lagrasse-%e2%9e%aa-ribaute)`
Et voici ce qui apparaît :![Sélection_812](/uploads/de69dea95ca97b076fc9c1e264ce06f6/Sélection_812.png)
En rouge sont entourés les tirets qui auraient dû être des virgules.
Cordialement,
Thierryhttps://framagit.org/framasoft/framadate/framadate/-/issues/388Mail delivery without authentication and encryption not working2018-11-12T07:49:50Zmt7479Mail delivery without authentication and encryption not workingI have a local installation of postfix configured to relay mail.
I've modified my config like this:
```
'auth' => false, // Enable SMTP authentication
'username' => '', // SMTP usern...I have a local installation of postfix configured to relay mail.
I've modified my config like this:
```
'auth' => false, // Enable SMTP authentication
'username' => '', // SMTP username
'password' => '', // SMTP password
'secure' => false, // Enable encryption (false, tls or ssl)
'port' => 25,
```
but Framadate still tries to send the mail with encryption:
```
cat /var/log/mail.log
Nov 11 21:52:15 xxx postfix/smtpd[7125]: lost connection after STARTTLS from localhost[127.0.0.1]
Nov 11 21:52:15 xxx postfix/smtpd[7125]: disconnect from localhost[127.0.0.1] ehlo=1 starttls=1 commands=2
```
I tried enclosing false in quotes without success.