Skip to content

Comments

Various fixes#833

Merged
SSilence merged 9 commits intofossar:masterfrom
niol:ppr/fixqueue
Feb 4, 2017
Merged

Various fixes#833
SSilence merged 9 commits intofossar:masterfrom
niol:ppr/fixqueue

Conversation

@niol
Copy link
Collaborator

@niol niol commented Dec 28, 2016

Various fixes detailed in the individual patches.

@lumiru
Copy link
Contributor

lumiru commented Dec 31, 2016

I think you at least need to add a IfModule on .htaccess (or modify install/upgrade documentation about Apache configuration) :

<IfModule mod_authz_core.c>
                        Require all granted
</IfModule>

lang_login_password=Mot de passe
lang_no_title=Sans titre
lang_error=Une erreur est survenue
lang_streamerror=Erreur lors du chargement de la liste, essayer à nouveau?
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing a space before ?

@niol
Copy link
Collaborator Author

niol commented Jan 2, 2017

Regarding the <IfModule mod_authz_core.c>, as a selfoss user, I'd prefer a 500 Internal Server Error rather than my password hash being exposed and the rule not being applied silently.

@lumiru
Copy link
Contributor

lumiru commented Jan 3, 2017

You make a point.

So you should add something about the minimum version in installation instructions (or as known issue).

'INSERT INTO version (version) VALUES (9);'
));
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this double closing bracket?

@SSilence
Copy link
Member

SSilence commented Feb 3, 2017

@niol may you send me a short email (tobias.zeising@aditu.de) that I have your email address?

@SSilence SSilence merged commit f323b7d into fossar:master Feb 4, 2017
@niol niol deleted the ppr/fixqueue branch February 6, 2017 14:39
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

Successfully merging this pull request may close these issues.

5 participants