Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nginx / Lua based approach to Open Graph tags #422

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from
+181 −5
Draft

Nginx / Lua based approach to Open Graph tags #422

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
f962740
client/nginx: increase timeouts
robobenklein Jul 9, 2021
b2800ad
client: using openresty for content rewriting
robobenklein Jul 10, 2021
44d28f3
client: og:site_name served via lua content script
robobenklein Jul 10, 2021
b4851ce
client: set og:url to the request_uri
robobenklein Jul 10, 2021
1f3c6ca
client: openresty-fat -> openresty (save 200MB+)
robobenklein Jul 10, 2021
7d90a73
client: initial og:title and og:image for posts
robobenklein Jul 10, 2021
3808061
client: og-tags fix for restricted posts
robobenklein Jul 10, 2021
3ec013b
client/lua: handle the api server being offline
robobenklein Jul 10, 2021
9fdeb29
client/lua: further development for post tags
robobenklein Jul 10, 2021
9f1bb53
client/lua: additional post tags
robobenklein Jul 10, 2021
b302ce2
client/lua: stricter matches and home tags
robobenklein Jul 10, 2021
d5fda34
client/lua: fix user meta tags
robobenklein Jul 10, 2021
372b40a
client/nginx: meta tags use X-Forwarded-Proto
robobenklein Jul 11, 2021
6e07c61
client/lua: wrap tags with error handler
robobenklein Jul 11, 2021
d94f5e9
client/lua: /user/* "User not found" title
robobenklein Jul 11, 2021
2637a4c
Merge remote-tracking branch 'origin/master' into robo/og-tags
robobenklein Sep 18, 2021
4332713
client: /post/id/* allowed to match for post tags
robobenklein Sep 18, 2021
22835fa
Merge remote-tracking branch 'origin/master' into robo/og-tags
robobenklein Aug 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
client: og-tags fix for restricted posts
  • Loading branch information
@robobenklein
robobenklein committed Jul 10, 2021
commit 38080611d5f52637835f1a8c9990b19ccce23cf4
9 changes: 7 additions & 2 deletions client/metatags.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,13 @@ add_meta_tag("og:url", ngx.var.external_host_url .. ngx.var.request_uri)

if ngx.var.request_uri_path:match('^/post') then
local post_info = cjson.decode((ngx.location.capture("/_internal_api"..ngx.var.request_uri_path)).body)
add_meta_tag("og:image", ngx.var.external_host_url .. '/' .. post_info.contentUrl)
add_meta_tag("og:title", server_info.config.name .. " - Post " .. post_info.id)
-- If no permission to access, fields will be nil, thus cannot be concat'd
if post_info.contentUrl then
add_meta_tag("og:image", ngx.var.external_host_url .. '/' .. post_info.contentUrl)
end
if post_info.id then
add_meta_tag("og:title", server_info.config.name .. " - Post " .. post_info.id)
end
end

local final_response = page_html.body:gsub("{{ generated_head_tags }}", additional_tags)
Expand Down