Skip to content

Commit

Permalink
DOC: Add notice about CSRF on browsers in documentation
Browse files Browse the repository at this point in the history
proposed by @bernhardreiter
  • Loading branch information
Sebastian Wagner committed Apr 27, 2020
1 parent 87df78c commit ec6567e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ See the changelog for a full list of changes.
The environment variable name was corrected from `INTELMQ_MANGER_CONTROLLER_CMD` to `INTELMQ_MANGAER_CONTROLLER_CMD` you might need to adapt your configuration.
The old name will be available until version 3.0.

Use IntelMQ Manager only from a browser that can only access internal, trusted sites. (Because CSRF development is under way, see [#111](github.com/certtools/intelmq/issues/111)).

2.1.0 (2019-10-15)
------------------
The environment variable name was corrected from `INTELMQ_MANGER_CONTROLER_CMD` to `INTELMQ_MANGER_CONTROLLER_CMD` you might need to adapt your configuration.
Expand Down
2 changes: 2 additions & 0 deletions docs/INSTALL.md
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,8 @@ The way the current version is written, anyone can send a POST request and chang

Therefore you will need authentication and SSL.

Use IntelMQ Manager only from a browser that can only access internal, trusted sites. (Because CSRF development is under way, see [#111](https://github.com/certtools/intelmq-manager/issues/111)).

In addition, intelmq currently stores plaintext passwords in its configuration files. These can be read via intelmq-manager.

**Never ever allow unencrypted, unauthenticated access to intelmq-manager**.
Expand Down

1 comment on commit ec6567e

@bernhardreiter
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool, maybe a comma after Because CSRF would help readability.

Please sign in to comment.