git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 3/3] gitweb: Add support to Link: tag

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 5, 2012, 22:29 UTC
Message-ID
<7vd349rger.fsf@alter.siamese.dyndns.org>
In-Reply-To
<1341370046-19913-3-git-send-email-namhyung@kernel.org>
Namhyung Kim <namhyung@kernel.org> writes:
Show 30 quoted lines
> The tip tree is the one of major subsystem tree in the
> Linux kernel project. On the tip tree, the Link: (or
> similar Buglink:) tag is used for tracking the original
> discussion or context. Since it's ususally in the S-o-b
> area, it'd be better using same style with others.
>
> Also as it tends to contain a message-id sent from git
> send-email, a part of the line would set a wrong hyperlink
> like [1]. Fix it by not using format_log_line_html().
>
> [1] git.kernel.org/?p=linux/kernel/git/tip/tip.git;a=commit;h=08942f6d5d992e9486b07653fd87ea8182a22fa0
>
> Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> ---
>  gitweb/gitweb.perl |   10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 362784d..3d6a705 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -4494,6 +4494,16 @@ sub git_print_log {
>  			next;
>  		}
>  
> +		if ($line =~ m,\s*([a-z]*link): (https?://\S+),i) {
> +			if (! $opts{'-remove_signoff'}) {
> +				print "<span class=\"signoff\">" . esc_html($1) . ": " .
> +					"<a href=\"" . esc_html($2) . "\">" . esc_html($2) . "</a>" .
> +					"</span><br/>\n";

Thanks. Is the first use of esc_html($2) correct (I am always confused between esc_html, esc_param and esc_url)?

Show 8 quoted lines
> +				$skip_blank_line = 1;
> +			}
> +			next;
> +		}
> +
>  		# print only one empty line
>  		# do not print empty line after signoff
>  		if ($line eq "") {
Previous: Namhyung KimNext: Junio C Hamano
Message 4 of 5 in “gitweb: Cleanup git_print_log()”
  1. 1/3 gitweb: Cleanup git_print_log()Namhyung Kim, Jul 4, 2012
  2. 2/3 gitweb: Handle other types of tag in git_print_logNamhyung Kim, Jul 4, 2012
  3. 3/3 gitweb: Add support to Link: tagNamhyung Kim, Jul 4, 2012
  4. Junio C HamanoJul 5, 2012
  5. Junio C HamanoJul 5, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.