Prepare 3.20190228 for future release

doc: Document security issues involving LWP::UserAgent

Recommend the LWPx::ParanoidAgent module where appropriate.
It is particularly important for openid, since unauthenticated users
can control which URLs that plugin will contact. Conversely, it is
non-critical for blogspam, since the URL to be contacted is under
the wiki administrator's control.

Signed-off-by: Simon McVittie <smcv@debian.org>

useragent: Automatically choose whether to use LWPx::ParanoidAgent

The simple implementation of this, which I'd prefer to use, would be:
if we can import LWPx::ParanoidAgent, use it; otherwise, use
LWP::UserAgent.

However, aggregate has historically worked with proxies, and
LWPx::ParanoidAgent quite reasonably refuses to work with proxies
(because it can't know whether those proxies are going to do the same
filtering that LWPx::ParanoidAgent would).

Signed-off-by: Simon McVittie <smcv@debian.org>

useragent: Don't allow non-HTTP protocols to be used

This prevents the aggregate plugin from being used to read the contents
of local files via file:/// URLs.

Signed-off-by: Simon McVittie <smcv@debian.org>

useragent: Raise an exception if the LWP module can't be loaded

Signed-off-by: Simon McVittie <smcv@debian.org>

po: Always filter .po files

The input to filter hooks is meant to be the content of a source file
on disk. If we only filter once per (page, destpage) pair, and a page
is inlined into the same destpage more than once, then the second
occurrence will render as the result of htmlizing .po source as if
it was Markdown (or whatever the type of the corresponding master page
is), which is never going to end well.

The alreadyfiltered mechanism was added in commit 1e874b3f to avoid
preprocessing loops, but I'm not sure where it could lead to a loop:
filter hooks are only called from IkiWiki::filter, which is only called
on page content from disk or on proposed content being previewed.
According to <https://bugs.debian.org/911356#41>, deleting the
alreadyfiltered mechanism resolves the problem, as well as simplifying
the code.

Closes: #911356
Tested-by: intrigeri

Recommend against cvsps3 (haven't tried it).

Announce v3.20190207

Signed-off-by: Simon McVittie <smcv@debian.org>

Prepare new release

Signed-off-by: Simon McVittie <smcv@debian.org>

review

comment

tag as reviewed

respond

Exclude reviewed patches from this list

close

append javascript after CSS

Javascript resources should be presented to browsers after CSS, and
"after the fold" (ATF) according to the best practices:

https://developers.google.com/speed/docs/insights/mobile#PutStylesBeforeScripts

This change allows the browser to download Javascript files in
parallel, by including Javascript on the *closing* </body> tag instead
of the opening tag.

We also improve the regex to tolerate spaces before the body tag, as
some templates have (proper) indentation for the tag.

Remove unreachable git repositories

Signed-off-by: Simon McVittie <smcv@debian.org>

Mark as applied

Signed-off-by: Simon McVittie <smcv@debian.org>

remove the "add comment" button from printed media

Add a missing changelog entry.

Update changelog

Signed-off-by: Simon McVittie <smcv@debian.org>

Allow Breezy as alternative to Bazaar.

(cherry picked from commit a07f048d9fc99928ebbb74b34f5d1932ff3d7884)

comments.t: Assert that comments get permalink metadata

Signed-off-by: Simon McVittie <smcv@debian.org>

comments.t: Exercise post-2009 comment naming

Since commit 6af6c89d, comments are in files whose names contain a hash.

Signed-off-by: Simon McVittie <smcv@debian.org>

old regexp would have failed for old comment page names

git-cgi.t: Exercise an alphanumeric, but non-ASCII, root page

My previous attempt to reproduce this bug used a non-alphanumeric
ASCII character. This is not currently considered to be a valid
value for rootpage, although for a "do what I mean" approach, perhaps
we should accept it and pass it through titlepage() or linkpage().

Using Chinese characters (which are considered to match [[:alnum:]]
even though the Chinese script is not, strictly speaking, an alphabet),
as in the original bug report, reproduces the bug.

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix inline plugin for non-ASCII rootpage

t: Exercise Chinese and Cyrillic page titles

Signed-off-by: Simon McVittie <smcv@debian.org>

trail: Allow unescaped punctuation in pagenames

By processing the pagenames through linkpage, we let users specify
page names that contain non-alphanumerics in a more natural way.

Signed-off-by: Simon McVittie <smcv@debian.org>

trail.t: Exercise numeric escapes in pagenames parameter

Signed-off-by: Simon McVittie <smcv@debian.org>

linkpage.t: Assert we can link to pages with literal underscore

Signed-off-by: Simon McVittie <smcv@debian.org>

t: Consistently remove temp directory before testing, not after

When a test fails, it's useful to be able to inspect the output.

Signed-off-by: Simon McVittie <smcv@debian.org>

build: Add `make reset-generated`

Signed-off-by: Simon McVittie <smcv@debian.org>

pagetitle.t, linkpage.t, titlepage.t: Exercise Unicode more

Signed-off-by: Simon McVittie <smcv@debian.org>

link to recently-added tests

I'm confused about what the bug is, and what's being fixed. Can you give a complete test or example?

close bug

Update changelog

Add a simple test for non-ASCII in tables

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix table plugin can handle UTF-8 csv format

respond

Fix syntax and escaping

git-cgi.t: Add a failing test for a blog with a non-ASCII rootpage

This is one of several possible bug reports on
"doc/bugs/About %2F problem" (I'm not sure what the actual bug being
reported is).

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Add a simple test for blog posts from a root page

This is the working base case for "doc/bugs/About %2F problem".

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Print query string as a TAP diagnostic

Signed-off-by: Simon McVittie <smcv@debian.org>

Reset example

This reverts commit 0b53772b99248d2868fac688cdaa227ad61ac08e

inline is another option of course

show tagging example

link to the map plugin, which i find more useful to show tagged pages

followup

fix permalink to comments

This commit was sponsored by Thomas May.

Merge branch 'master' of ssh://git.ikiwiki.info

bug report

User link and description changed

markdown backquotes

followup

de-indenting

example for inlining only one level of subpages

fail to inject IkiWiki::showform

patch works for me

ack

rename and remove plug can not work well with gettext

Suggest let table plugin support a kind of longtable

Add a patch deal with UTF-8 csv table's problem

Add a patch decode_utf8 inline's root page

add redirection

update for rename of bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt to bugs/po:_second_or_subsequent_inline_of_translated_page_inlines_.po_file__44___not_translated_content/20180628-patch.txt

rename bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn to bugs/po:_second_or_subsequent_inline_of_translated_page_inlines_.po_file__44___not_translated_content.mdwn

update for rename of todo/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt to bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt

rename todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn to bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn

refer to a branch

ikiwiki.info doesn't have fenced code blocks yet

Escape directives in code

Review

po: Improve test coverage

Signed-off-by: Simon McVittie <smcv@debian.org>

po: Add failing test for Debian bug #911356

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix broken link in basewiki

Links from pages in the basewiki (which is also a subset of ikiwiki's
own wiki) to pages on ikiwiki's own wiki have to use [[!iki]] so that
they will not become broken links when the page in question is copied
into other installations.

Detected by ./t/basewiki_brokenlinks.t.

Signed-off-by: Simon McVittie <smcv@debian.org>

I don't think the Chinese text causes this

comments (no review yet)

Revert spam

This reverts commit c600ed0e710d82601acf5ae4005b940f5cba8c08

related issue

error output when trying

rename postvote to postlink

This better explains what it contains, which is a wikilink to the page
to go to after posting the vote. And postlink is more consistent a name
with posttrail.

poll: Added postvote and posttrail options for better multi-page polls.

Modern web users probably expect the poll to move on automatically to the
next question, and this allows for that behavior.

Note that bestlink() runs at vote time, which avoids needing to make the
page containing the poll depend on the page that sets up a trail, as the
current trail at vote time will be used.

This commit was sponsored by Eric Drechsel on Patreon.

avoiding ban?

update my entry