Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc fixes #1262

Merged
merged 13 commits into from
Jul 6, 2022
Merged

doc fixes #1262

merged 13 commits into from
Jul 6, 2022

Conversation

peter-
Copy link
Contributor

@peter- peter- commented May 31, 2022

Mostly no longer working URLs (e.g. due to the SimpleSAMLphp web site changes), http links where https could be used, broken relative documentation URLs, etc.
(I started out with very targeted commits but ultimately lumpted together more changes in cf60ad2 since you may want to squash all those on merge anyway.)

The docs are still a big mess IMHO but now with fewer broken links.

Also note that https://filesender.wordpress.com/ still exists but hasn't been updated in years.
(The same goes for https://twitter.com/filesender which I left in docs/socialmedia.md for now, otherwise the page would essentially be empty after having removed references to the no longer functional http://blog.filesender.org/.)

@github-actions
Copy link

If there are selenium UI results for this code they will be at filesenderuici@cf60ad2

@peter- peter- mentioned this pull request May 31, 2022
@github-actions
Copy link

If there are selenium UI results for this code they will be at filesenderuici@e7fcb7e


Extract SimpleSAMLphp in a suitable directory and create symlink:

```
mkdir -p /opt/filesender
cd /opt/filesender
tar xvzf /root/src/simplesamlphp-1.15.4.tar.gz
ln -s simplesamlphp-1.15.4 simplesaml
tar xvzf /tmp/simplesamlphp-1.17.1.tar.gz
Copy link
Collaborator

Choose a reason for hiding this comment

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

We're at 1.19.5 already :)

@monkeyiq
Copy link
Contributor

monkeyiq commented Jul 5, 2022

Thank you for the PR :)

Only one thing caught my attention while reading over things... I am not sure why session_cookie_path was removed in this PR. It seems to be still referenced in https://github.com/filesender/filesender/blob/development/includes/init/init_web.php#L45
It may be that the existing docs for that are slightly off but perhaps leaving that directive in for now and changing the docs for it separately is a good idea.

Other than that (which I am happy to bring back post merge) everything looks good.

@monkeyiq monkeyiq merged commit 2125856 into filesender:development Jul 6, 2022
@monkeyiq
Copy link
Contributor

monkeyiq commented Jul 6, 2022

Hmm, I see now that we went from 2 docs for session_cookie_path down to the correct one doc stanza for session_cookie_path :)

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.

3 participants