fsf changes, meant to be rebased on upstream

As upstream changes, rebase this commit on top.

The change to read_body.php, documented here
https://sourceforge.net/p/squirrelmail/mailman/message/36992407/
copied here in case that link dies:

In the svn version, when viewing a message, if you click the move or
copy button, the page changes to the list of messages. In older versions
such as 1.4.13, that button was enabled by the delete_move_next plugin,
and when pressing move, the page would change to the next message. That
is clearly the way it should work. When using the preview panel, this
behavior especially makes no sense because you were already seeing the
message list and it just turns the preview panel into blank space.

After a quick look into the code, I was able to make a one line change
to the svn version so that the the page does not change at all when
pressing move, which helps workaround the problem so that I can then
press a key to go to the next message. I'm hoping someone here is
familiar with the code and can help identify the root cause and find a
proper fix so it goes to the next message.

Here is a comparison of the post request when clicking move in svn vs
old working 1.4.13:

SVN post:

https://webmail.fsf.org/src/read_body.php?mailbox=INBOX&sort=6&startMessage=1&passed_id=18
form data:
show_more "0"
move_id "19"
targetMailbox "INBOX.Trash"

Working 1.4.13 post:

https://mail1p.fsf.org/src/right_main.php
form data:
smtoken "EEZR1hS4cPUh"
mailbox "INBOX"
msg[0] "19"
targetMailbox "INBOX.Trash"
moveButton "Move"

I figured out that if the post url didn't have any path, or had a path
of "right_main.php", like the old version, it would stop changing the
page, and there was already an example in the code of not passing the
path, so I copied that to make this hacky patch.

I enabled level 2 debug mode, didnt result in any output in the logs.

Configtest shows no errors.

Fix PHP warning

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@15015 7612ce4b-ef26-0410-bec9-ea0150e637f0

Hack for problem where $i is used as both boolean and int

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@15013 7612ce4b-ef26-0410-bec9-ea0150e637f0

Happy New Year

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@15008 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix: Don't feed an integer to sizeof()

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@15001 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix broken parenthesis

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14999 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add TODO

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14998 7612ce4b-ef26-0410-bec9-ea0150e637f0

Test option page name, don't assign it

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14996 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove extraneous reset

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14995 7612ce4b-ef26-0410-bec9-ea0150e637f0

Explicitly declare formerly dynamic class property (deprecated PHP behavior)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14994 7612ce4b-ef26-0410-bec9-ea0150e637f0

Don't str_replace() on null values

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14990 7612ce4b-ef26-0410-bec9-ea0150e637f0

Make sure $prefs_cache is initialized when resetting the cache

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14988 7612ce4b-ef26-0410-bec9-ea0150e637f0

Happy New Year

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14986 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix poorly written timezone parsing

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14979 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove use of each()

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14977 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix PHP notice

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14973 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove use of each() and make sense of poorly named variables

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14971 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix undefined variable notices

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14970 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add ability to explicitly set address book DB connection charset, though usually if database defaults are sensible this is not something to be concerned with in configuring SquirrelMail

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14969 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add ability to explicitly set prefs DB connection charset, though usually if database defaults are sensible this is not something to be concerned with in configuring SquirrelMail

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14968 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix PHP notice when property doesn't exist

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14965 7612ce4b-ef26-0410-bec9-ea0150e637f0

Greatly fix the plaintext display of messages that do not have a text part. Patch from Alexey Shpakovsky (#496)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14961 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix: Login mechanisms other than "login" were broken if server returned untagged data before login response (#2854)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14959 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add option to folder prefs for marking deleted messages as read

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14957 7612ce4b-ef26-0410-bec9-ea0150e637f0

Implement SQL identifier quoting in all cases (alleviates issue double-quoting user field under PGSQL, #177)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14954 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix PHP notice when property doesn't exist (#2863)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14952 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix: When "Return To Message List After Move" is off, deleting message from message view screen was broken (#2863)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14951 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix bug where could not toggle flag (delete, etc) a single message

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14950 7612ce4b-ef26-0410-bec9-ea0150e637f0

Cause message subject to show in page title when message view is loaded in its own window/tab

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14949 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add proper RFC 3461 DSN functionality (previously we relied only on the Return-Receipt-To header)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14947 7612ce4b-ef26-0410-bec9-ea0150e637f0

Sort out DSN functionality

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14946 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add ability for plugins to request additional header fields in the message list lookup

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14942 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix whacky theme name logic, though it's still needlessly complex and should not be using 'none' when no theme is chosen...

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14940 7612ce4b-ef26-0410-bec9-ea0150e637f0

Syntax fix (thanks to Andrew Sachen)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14939 7612ce4b-ef26-0410-bec9-ea0150e637f0

Happy New Year

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14938 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix mixed use of message ID list (#2859)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14936 7612ce4b-ef26-0410-bec9-ea0150e637f0

Mild improvement in plugin detection

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14935 7612ce4b-ef26-0410-bec9-ea0150e637f0

Expand clickability of column sort links

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14933 7612ce4b-ef26-0410-bec9-ea0150e637f0

Someone meant to use strlen() instead of sizeof() but neither is necessary

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14932 7612ce4b-ef26-0410-bec9-ea0150e637f0

Error context is removed from PHP 8

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14931 7612ce4b-ef26-0410-bec9-ea0150e637f0

Someone added non-optional parameter after optional one; it's not optional, but we can fix it like this

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14930 7612ce4b-ef26-0410-bec9-ea0150e637f0

Forgot to change constructor name

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14929 7612ce4b-ef26-0410-bec9-ea0150e637f0

Class constructor updates that were missed previously

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14928 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix certain messages with headers in unknown charsets encoded as quoted printable showing up as blank sender/subject in message list. There are two ways to fix it, and the more accurate but costly one is left unfinished since the easy fix seems to have helped in every case I know of

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14927 7612ce4b-ef26-0410-bec9-ea0150e637f0

Minor, rare bug fix: Properly reduce multiple spaces in date headers

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14925 7612ce4b-ef26-0410-bec9-ea0150e637f0

Make sure drafts are sent with the current date and not that of its previous 'Save Draft' action

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14922 7612ce4b-ef26-0410-bec9-ea0150e637f0

Browser seems to respond better when deleting the cookie if you void its value also; helps actually get the cookie to be removed even though this feels more like a browser problem... maybe it does not like a date in 1970?

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14920 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add cookie SameSite attribute; uses default if "Strict" but can be overridden by $same_site_cookies in config_local.php

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14918 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix order of optional args

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14915 7612ce4b-ef26-0410-bec9-ea0150e637f0

Allow plugins to add non-link text for attachments

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14913 7612ce4b-ef26-0410-bec9-ea0150e637f0

Scale images to viewable space

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14911 7612ce4b-ef26-0410-bec9-ea0150e637f0

Synchronize optional parameters

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14908 7612ce4b-ef26-0410-bec9-ea0150e637f0

Make sure large images fit in viewable space

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14907 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fixing layout issues with some HTML messages. Please provide feedback (and examples to reproduce) if this causes issues

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14905 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix a couple functions with required parameters after optional ones

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14903 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove stray curly brackets

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14902 7612ce4b-ef26-0410-bec9-ea0150e637f0

Migrate away from using create_function as long as PHP 5.3+ is available

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14900 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove random default argument value in the middle of argument list

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14898 7612ce4b-ef26-0410-bec9-ea0150e637f0

Stop using curly braces

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14894 7612ce4b-ef26-0410-bec9-ea0150e637f0

Stop using curly braces

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14893 7612ce4b-ef26-0410-bec9-ea0150e637f0

Stop using curly braces

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14892 7612ce4b-ef26-0410-bec9-ea0150e637f0

Happy New Year

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14886 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix variable name typo

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14884 7612ce4b-ef26-0410-bec9-ea0150e637f0

Typo fix

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14882 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove use of each() as it is deprecated

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14881 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix for unknown broken mailer using "UTC" in Date header

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14878 7612ce4b-ef26-0410-bec9-ea0150e637f0

Give attribution

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14877 7612ce4b-ef26-0410-bec9-ea0150e637f0

PHP 8 compatibility fix

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14876 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add default ability to view calendar attachments sent as application/ics

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14875 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix what I think was a misunderstanding of what the "extra" attribute was supposed to be for the anchor links for attachment actions

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14873 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix index value that was left over from 1.4.x

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14871 7612ce4b-ef26-0410-bec9-ea0150e637f0

Allow plugins to provide their own view/download attachment links

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14870 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix previously incomplete commit

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14869 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add MIME type parameters ("type0", "type1") to the end of the plugin arguments for all attachment hooks

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14867 7612ce4b-ef26-0410-bec9-ea0150e637f0

Remove needless code

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14866 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix overlooked default parameter value

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14864 7612ce4b-ef26-0410-bec9-ea0150e637f0

More accurate filesizes

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14863 7612ce4b-ef26-0410-bec9-ea0150e637f0

Show better filesize approximations for base64-encoded attachments

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14862 7612ce4b-ef26-0410-bec9-ea0150e637f0

Uploaded file sizes seem to be reported differently by PHP filesize(); changing divisor from 1024 to 1000 in this case. You can set $upload_filesize_divisor in config/config_local.php to 1024 if this breaks things for you. Feedback appreciated for this one.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14861 7612ce4b-ef26-0410-bec9-ea0150e637f0

Implement ability to move to next message after moving the one you are viewing, per the way delete_move_next plugins does from v1.4.x

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14856 7612ce4b-ef26-0410-bec9-ea0150e637f0

Document $php_self_pattern and $php_self_replacement

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14853 7612ce4b-ef26-0410-bec9-ea0150e637f0

Document $php_self_pattern and $php_self_replacement

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14851 7612ce4b-ef26-0410-bec9-ea0150e637f0

Cache $PHP_SELF value, add ability to make custom changes to $PHP_SELF by putting $php_self_pattern and $php_self_replacement in config/config_local.php

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14850 7612ce4b-ef26-0410-bec9-ea0150e637f0

Search page should remember last move-target too

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14848 7612ce4b-ef26-0410-bec9-ea0150e637f0

Happy New Year

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14845 7612ce4b-ef26-0410-bec9-ea0150e637f0

Even better HTML email content style containment

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14839 7612ce4b-ef26-0410-bec9-ea0150e637f0

Better HTML email content style containment

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14837 7612ce4b-ef26-0410-bec9-ea0150e637f0

Fix broken anchor links

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14834 7612ce4b-ef26-0410-bec9-ea0150e637f0

Document CVE-2019-12970 fix

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14831 7612ce4b-ef26-0410-bec9-ea0150e637f0

Document new config_local item for CVE-2019-12970

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14830 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add handling for RCDATA and RAWTEXT elements in HTML sanitizer (CVE-2019-12970)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14829 7612ce4b-ef26-0410-bec9-ea0150e637f0

PHP7.2 fix (#2848)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14826 7612ce4b-ef26-0410-bec9-ea0150e637f0

Some browswers were not putting cursor at beginning of message body after focus

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14824 7612ce4b-ef26-0410-bec9-ea0150e637f0

Don't wrap headers right after the name (configurable)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14822 7612ce4b-ef26-0410-bec9-ea0150e637f0

Correct mistaken use of rfc822_header->date field that was being treated as a date string when it is only ever a timestamp

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14818 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add IMAP ID command (RFC2971), sent after every login - use by setting $imap_id_command_args in config/config_local.php (see notes in config/config_local.example.php for more details)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14814 7612ce4b-ef26-0410-bec9-ea0150e637f0

Updated SVG handling, closing several related vulnerabilities reported in #2831 and CVE-2018-14950, CVE-2018-14951, CVE-2018-14952, CVE-2018-14953, CVE-2018-14954, CVE-2018-14955

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14810 7612ce4b-ef26-0410-bec9-ea0150e637f0

Add new options for SVG handling and broken base64-encoded messages

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14809 7612ce4b-ef26-0410-bec9-ea0150e637f0

Disable SVG display be default

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14807 7612ce4b-ef26-0410-bec9-ea0150e637f0

Updated SVG handling, gracefully fix broken base64-encoded messages, also close XSS reported in #2831 and CVE-2018-14950, CVE-2018-14951, CVE-2018-14952, CVE-2018-14953, CVE-2018-14954, CVE-2018-14955

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14805 7612ce4b-ef26-0410-bec9-ea0150e637f0