With a dynamic configuration, it is not desirable to load from literal files
By storing a configuration object against a filepath, subsequent calls to
getConfig work transparently, bypassing the disk load step.

Updated config.php template to include comma after baseURL

Fixes https://github.com/simplesamlphp/simplesamlphp/issues/818

* Check responseHeaders presence before use them

And so avoid some messages in log file.

This is minor, but if a comma is required when this is enabled the template should reflect that.

Document the new Twig templating system

It is deprecated as of PHP 7.2

Edits to CONTRIBUTING.md

This marker is used by some Markdown compilers to start the Table of
Contents hierarchy at this point.

Per suggestion from @tvdijen

Add some basic documentation on how the new Twig templating system works.

PSR-2 compliant whitespace in error classes

space is required before auth data

forward-port of my recent fix in branch 1.15

Fixes issue https://github.com/simplesamlphp/simplesamlphp/issues/793

It was already added under 1.15

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This section felt a little like a blob of good, unorganized thoughts. I
hope this is a little easier to read and understand. It will be easier
for someone to go to this document and point to a specific section now.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This is a general readability improvement. The meaning of these
paragraphs has not changed.

This comment is quite arbitrary in the view of the general public. Not
sure if anyone would know what the acronym "T.O.C." means.

The random line breaks everywhere in this file makes it hard to read at
narrow or wide window sizes whether you are using a Markdown reader or
just reading the markdown itself.

This puts all of the paragraphs and list items on their own line. The
one reading this file can decide for themselves how narrow or wide they
want the text to be displayed at.

I have also fixed the indentaiton of list items. Markdown will only read
a list item as "indented" when you use 4 spaces.

My final change in this commit was to remove all of the new lines below
headers. After reading so many markdown files, I find it much more
readable when there is one new line _before_ the header and zero new
lines below it.

docs: Update reference to `redirect.trustedsites`. Fixes #786

doc: Switch to namespaced SimpleSAML\Module.

SAML: Fix typo in logged error message