[PR #63] [MERGED] add http lua directives #67

New issue

Closed

opened 2026-02-28 01:20:45 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 01:20:45 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/tufanbarisyildirim/gonginx/pull/63
Author: @hjweddie
Created: 1/20/2025
Status: ✅ Merged
Merged: 1/20/2025
Merged by: @tufanbarisyildirim

Base: master ← Head: master

📝 Commits (1)

a896b5d add http lua directives

📊 Changes

1 file changed (+84 additions, -2 deletions)

View changed files

📝 parser/valid_directives.go (+84 -2)

📄 Description

We use gonginx to do much configuration static check in our production environment, also we use openresty which embeds lua-nginx-modules

We find that parser may parse fail with error message "unknown directive 'xxx_lua_xxx'", hope that gonginx could support directive in openresty

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/tufanbarisyildirim/gonginx/pull/63 **Author:** [@hjweddie](https://github.com/hjweddie) **Created:** 1/20/2025 **Status:** ✅ Merged **Merged:** 1/20/2025 **Merged by:** [@tufanbarisyildirim](https://github.com/tufanbarisyildirim) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`a896b5d`](https://github.com/tufanbarisyildirim/gonginx/commit/a896b5d9c604cf4be0cb504ea9ddec7ba9040f59) add http lua directives ### 📊 Changes **1 file changed** (+84 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `parser/valid_directives.go` (+84 -2) </details> ### 📄 Description We use gonginx to do much configuration static check in our production environment, also we use openresty which embeds lua-nginx-modules We find that parser may parse fail with error message "unknown directive 'xxx_lua_xxx'", hope that gonginx could support directive in openresty --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>