[PATCH] address: filter out domain from address-as-name idents

2016-06-30 Thread Eric Wong
It seems common for address entries to end up as: "foo@example" Avoid needlessly displaying the domain name in that case. --- lib/PublicInbox/Address.pm | 4 +++- t/address.t| 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/Address.pm b

[PATCH] examples/README: update URL to point to our HTTPS backend

2016-06-30 Thread Eric Wong
It's browseable, too! --- examples/README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/README b/examples/README index 1244cb2..1d5dcd3 100644 --- a/examples/README +++ b/examples/README @@ -16,4 +16,4 @@ apache2_perl.conf - intended to be the basis of a production

[PATCH] t/watch_maildir: quiet down spam check warning

2016-06-30 Thread Eric Wong
Probably better than bloating our own API with configurable warning streams and such... --- t/watch_maildir.t | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/t/watch_maildir.t b/t/watch_maildir.t index 2138963..3969c80 100644 --- a/t/watch_maildir.t +++ b/t/watch_maildir.t

[PATCH 4/2] www_stream: fix stupid typo :x

2016-06-30 Thread Eric Wong
Note to self: remember to run tests Fixes: 52052329aced ("git: allow cloning from the URL root, too") --- lib/PublicInbox/WwwStream.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/PublicInbox/WwwStream.pm b/lib/PublicInbox/WwwStream.pm index 87a461e..fdab4da 100644 ---

[PATCH 3/2] git: allow cloning from the URL root, too

2016-06-30 Thread Eric Wong
This means we can still show non-git users a somewhat browseable URL with a link to the README.html file while allowing git users to type less when cloning. All of the following are supported: git clone https://public-inbox.org/ public-inbox git clone https://public-inbox.org/public-inbox git clo

[PATCH 2/2] update git repo location to https:// using GitHTTPBackend.pm

2016-06-30 Thread Eric Wong
Might as well eat our own dogfood... --- Documentation/dc-dlvr-spam-flow.txt | 2 +- README | 2 +- examples/public-inbox.psgi | 14 +- examples/unsubscribe.psgi | 3 ++- lib/PublicInbox/Unsubscribe.pm | 2 +- lib/PublicInbox/Www

[PATCH 0/2] HTTPS smart git repository for our code

2016-06-30 Thread Eric Wong
Eating our own dogfood, git://80x24.org/public-inbox remains available for those without curl or up-to-date TLS stack/certs. Eric Wong (2): githttpbackend: allow git to be a regular scalar string update git repo location to https:// using GitHTTPBackend.pm Documentation/dc-dlvr-spam-

[PATCH 1/2] githttpbackend: allow git to be a regular scalar string

2016-06-30 Thread Eric Wong
No point in forcing users to pass a hashref/object to get a single git directory. --- lib/PublicInbox/GitHTTPBackend.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/PublicInbox/GitHTTPBackend.pm b/lib/PublicInbox/GitHTTPBackend.pm index 7267a1d..4f58c6b 100644 --- a

[PATCH] searchview: add missing newline in search results

2016-06-30 Thread Eric Wong
Hrm... is there a more obvious way to do an internal API for this while still being streamable? --- lib/PublicInbox/SearchView.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index d019a0f..ce1eff1 100644 --- a/

[PATCH] www_stream: fix search for new.html endpoint

2016-06-30 Thread Eric Wong
We want to avoid the bare './' wherever possible, but it doesn't seem possible here. --- lib/PublicInbox/WwwStream.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/WwwStream.pm b/lib/PublicInbox/WwwStream.pm index 6de1b31..d9abb5a 100644 --- a/lib/PublicInbo

[PATCH 14/13] view: fix permalink and raw links at the top

2016-06-30 Thread Eric Wong
Oops :x I really need to whip check-inbox.perl into shape or at least start running it, again. Fixes: e29518088b3f ("view: fix up some HTML injection via Message-ID vectors") --- lib/PublicInbox/View.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/PublicInbox/View.

[PATCH] view: move per-message anchor to top in conversation view

2016-06-30 Thread Eric Wong
This fixes the '^' (up) link in the $INBOX/new.html endpoint for search-less displays. --- lib/PublicInbox/View.pm | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 8487c2a..11d8dd5 100644 --- a/lib/PublicInbox/Vi

[PATCH] searchview: show result count in thread index, for now

2016-06-30 Thread Eric Wong
I'm not sure what to show here, actually; but it's better than triggering an uninitialized variable warning. --- lib/PublicInbox/SearchView.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index 30a310c..d019a0f 100644 --- a/lib/P

[PATCH 1/2] view: improve readability of msg_reply with here-doc

2016-06-30 Thread Eric Wong
This should make formatting more apparent since we can rely on semantics. --- lib/PublicInbox/View.pm | 36 ++-- 1 file changed, 22 insertions(+), 14 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 27dd155..9d48dfc 100644 --- a/li

[PATCH 0/2] view: reply instruction tweaks

2016-06-30 Thread Eric Wong
It may be 2016, but top-posting still makes me cringe :< Eric Wong (2): view: improve readability of msg_reply with here-doc view: reference posting style article on Wikipedia lib/PublicInbox/View.pm | 42 -- 1 file changed, 28 insertions(+), 1

[PATCH 2/2] view: reference posting style article on Wikipedia

2016-06-30 Thread Eric Wong
Storage is precious when it is forever and distributed. And public-inbox aims to not only end Eternal September(*), but to build a world less-centralized than Usenet ever was: forkable discussion groups (*) https://en.wikipedia.org/wiki/Eternal_September --- lib/PublicInbox/View.pm | 6 ++ 1

[PATCH 10/13] view: default to flat/hybrid thread display

2016-06-30 Thread Eric Wong
This is friendlier for people on small screens and usually eliminates the need to scroll horizontally. --- lib/PublicInbox/View.pm | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 44130b9..22d7250 100644 --- a/lib/

[PATCH 13/13] www_stream: add response wrapper sub

2016-06-30 Thread Eric Wong
This encapsulates an entire PSGI response array, hopefully making it easier to generate responses and avoid typos when setting the Content-Type. --- lib/PublicInbox/Feed.pm | 3 +-- lib/PublicInbox/SearchView.pm | 4 +--- lib/PublicInbox/View.pm | 8 +++- lib/PublicInbox/WWW.pm

[PATCH 11/13] view: show thread size when linking to summary

2016-06-30 Thread Eric Wong
This should give readers a better idea of what to expect. --- lib/PublicInbox/View.pm | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 22d7250..fac53eb 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.p

[PATCH 05/13] feed: add $INBOX/new.html endpoint

2016-06-30 Thread Eric Wong
This acts like the Atom feed; but should be viewable directly from browsers. --- lib/PublicInbox/Feed.pm | 27 +++ lib/PublicInbox/WWW.pm | 10 +- 2 files changed, 36 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/Feed.pm b/lib/PublicInbox/Feed.pm index

[PATCH 01/13] www: implement hybrid flat+thread conversation view

2016-06-30 Thread Eric Wong
This should be more accessible to readers on narrow terminals (or giant fonts) while providing a chronological view which is also aware of message threading relationships. --- TODO | 2 - lib/PublicInbox/SearchView.pm | 42 +++ lib/PublicInbox/View.pm | 248 ++

[PATCH 04/13] view: merge $state hash with existing $ctx

2016-06-30 Thread Eric Wong
This reduces the level of indirection to reach certain objects within the hash and there are no namespace or lifetime conflicts anyways. --- lib/PublicInbox/Feed.pm | 25 +++ lib/PublicInbox/SearchView.pm | 25 +++ lib/PublicInbox/View.pm | 149 ---

[PATCH 07/13] view: show more nearby messages in flat thread view

2016-06-30 Thread Eric Wong
Context is important, but so is conserving precious screen space. Decisions :< --- lib/PublicInbox/View.pm | 42 -- 1 file changed, 36 insertions(+), 6 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 9393d44..0b96bda 10064

[PATCH 12/13] view: fixup bad reference to new_msgid

2016-06-30 Thread Eric Wong
Oops, this endpoint needs testing :x --- lib/PublicInbox/View.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index db2bd20..b4f80d1 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -129,7 +129,7 @@ sub i

[PATCH 02/13] www: use WwwStream for dumping thread and search views

2016-06-30 Thread Eric Wong
This allows us the HTTP server to react to backpressure from slow clients when writing. As a side effect, this also makes it easier for us to maintain a consistent header/footer across our HTML. --- lib/PublicInbox/Feed.pm | 3 +- lib/PublicInbox/SearchView.pm | 106 +-

[PATCH 06/13] view: tweak thread/index header slightly

2016-06-30 Thread Eric Wong
This makes the top permalink/raw as well as the In-Reply-To show up without search. While we're at it, try to make the links on the thread index from the "X siblings, Y replies" more obvious. --- lib/PublicInbox/View.pm | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff -

[PATCH 09/13] view: fix up some HTML injection via Message-ID vectors

2016-06-30 Thread Eric Wong
Oops, these were only introduced during the hybrid flat thread view reworking and never deployed. --- lib/PublicInbox/View.pm | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 17d6de5..44130b9 100644 --- a

[PATCH 03/13] view: show thread context in the thread-aware flat view

2016-06-30 Thread Eric Wong
This lets user have a small window of the context of the current message relative to other threads. --- lib/PublicInbox/Feed.pm | 3 +- lib/PublicInbox/SearchView.pm | 2 +- lib/PublicInbox/View.pm | 160 +- 3 files changed, 100 insertions(+)

[PATCH 08/13] www: reinstate old thread view as an option

2016-06-30 Thread Eric Wong
This hybrid view is better than the old flat, but can still fall down compared to the old threaded view in some cases. --- lib/PublicInbox/View.pm | 109 lib/PublicInbox/WWW.pm | 13 +++--- t/plack.t | 2 +- 3 files changed, 109 in

[PATCH 0/13] www: hybrid flat+thread conversation view

2016-06-30 Thread Eric Wong
I've been long-dreaming of this and finally it's at least publishable (I hope :x). This flat view with thread skeletons is 100% more usable than the dumb old one, but a little slower (naturally :<) I was originally hoping to remove the threaded /t/ endpoint conversation view entirely to reduce se

[PATCH] Revert "view: update git-send-email URL"

2016-06-30 Thread Eric Wong
This reverts commit a391cf5aaf7181b5e5e20eb240c7ce50cbdf8fa2 as kernel.org is updating again. Ref: http://mid.gmane.org/xmqqpor18a2n@gitster.mtv.corp.google.com --- lib/PublicInbox/View.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/View.pm b/lib/Publ