user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 1/5] wwwhighlight: do linkification, too
  2019-04-24 23:02  7% [PATCH 0/5] cgit + WwwHighlight.pm integration Eric Wong
@ 2019-04-24 23:02  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2019-04-24 23:02 UTC (permalink / raw)
  To: meta

Otherwise, there's no reason to use this API over highlight(1).
Maybe this can be an option in the future; but I'm struggling to
find a reason to not do it by default.
---
 lib/PublicInbox/WwwHighlight.pm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/PublicInbox/WwwHighlight.pm b/lib/PublicInbox/WwwHighlight.pm
index 0191640..d8101b7 100644
--- a/lib/PublicInbox/WwwHighlight.pm
+++ b/lib/PublicInbox/WwwHighlight.pm
@@ -24,6 +24,7 @@ use warnings;
 use bytes (); # only for bytes::length
 use HTTP::Status qw(status_message);
 use parent qw(PublicInbox::HlMod);
+use PublicInbox::Linkify qw();
 
 # TODO: support highlight(1) for distros which don't package the
 # SWIG extension.  Also, there may be admins who don't want to
@@ -64,7 +65,10 @@ sub call {
 	return r(405) if $req_method ne 'PUT';
 
 	my $bref = read_in_full($env) or return r(500);
+	my $l = PublicInbox::Linkify->new;
+	$l->linkify_1($$bref);
 	$bref = $self->do_hl($bref, $env->{PATH_INFO});
+	$l->linkify_2($$bref);
 
 	my $h = [ 'Content-Type', 'text/html; charset=UTF-8' ];
 	push @$h, 'Content-Length', bytes::length($$bref);
-- 
EW


^ permalink raw reply related	[relevance 7%]

* [PATCH 0/5] cgit + WwwHighlight.pm integration
@ 2019-04-24 23:02  7% Eric Wong
  2019-04-24 23:02  7% ` [PATCH 1/5] wwwhighlight: do linkification, too Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2019-04-24 23:02 UTC (permalink / raw)
  To: meta

cgit-wwwhighlight-filter.lua can benefit cgit installations by
allowing them to avoid spawning a new process for about-filter
and source-filter in cgitrc.

Instead of spawning highlight(1) or similar every time a cgit
filter is invoked, we can have a long-lived PSGI server running
WwwHighlight.pm(*) and rely on lua-http to access the server
over loopback.

WwwHighlight.pm is also updated to support the same
linkification logic we use for emails.


(*) see examples/highlight.psgi

Eric Wong (5):
  wwwhighlight: do linkification, too
  examples/cgit-commit-filter: remove unused variable
  wwwhighlight: deal with no-op highlight
  examples: cgit filter for use with WwwHighlight
  examples/cgit-commit-filter.lua: some doc updates

 MANIFEST                              |  1 +
 examples/cgit-commit-filter.lua       | 10 ++++-
 examples/cgit-wwwhighlight-filter.lua | 57 +++++++++++++++++++++++++++
 lib/PublicInbox/WwwHighlight.pm       | 11 +++++-
 4 files changed, 77 insertions(+), 2 deletions(-)
 create mode 100644 examples/cgit-wwwhighlight-filter.lua

^ permalink raw reply	[relevance 7%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2019-04-24 23:02  7% [PATCH 0/5] cgit + WwwHighlight.pm integration Eric Wong
2019-04-24 23:02  7% ` [PATCH 1/5] wwwhighlight: do linkification, too Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).