[PR #7] [MERGED] Recursive include parsing #36

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/tufanbarisyildirim/gonginx/pull/7
Author: @riadafridishibly
Created: 4/6/2022
Status: ✅ Merged
Merged: 5/31/2022
Merged by: @tufanbarisyildirim

Base: master ← Head: feature/support-recursive-include-parsing

📝 Commits (9)

eb064c2 Add parse options
81a37ad Parse includes recursively
8fcd03a Find and Get directives from includes
c45fe25 Include parsing tests
06a0a4e Update testdata
91c76bb Add dumper for include
b7a7740 Paser include guard
f3fcffe update deps
bbe2c0b check err first

📊 Changes

17 files changed (+247 additions, -7 deletions)

View changed files

📝 block.go (+5 -0)
📝 dumper.go (+49 -0)
📝 go.mod (+5 -1)
📝 go.sum (+26 -0)
📝 http.go (+5 -0)
📝 include.go (+18 -0)
📝 parser/parser.go (+97 -6)
📝 parser/parser_test.go (+9 -0)
➕ testdata/include-glob/conf.d/location-letsencrypt.conf (+4 -0)
➕ testdata/include-glob/conf.d/location-root.conf (+3 -0)
➕ testdata/include-glob/events.conf (+3 -0)
➕ testdata/include-glob/http.conf (+2 -0)
➕ testdata/include-glob/nginx.conf (+5 -0)
➕ testdata/include-glob/sites-enabled/conf.d/location-letsencrypt.conf (+2 -0)
➕ testdata/include-glob/sites-enabled/conf.d/location-root.conf (+2 -0)
➕ testdata/include-glob/sites-enabled/example.com.conf (+6 -0)
➕ testdata/include-glob/sites-enabled/mysite.com.conf (+6 -0)

📄 Description

Parse included files recursively

Parses the include directives. And lookup directive in the included configs.

_{🔄 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/7 **Author:** [@riadafridishibly](https://github.com/riadafridishibly) **Created:** 4/6/2022 **Status:** ✅ Merged **Merged:** 5/31/2022 **Merged by:** [@tufanbarisyildirim](https://github.com/tufanbarisyildirim) **Base:** `master` ← **Head:** `feature/support-recursive-include-parsing` --- ### 📝 Commits (9) - [`eb064c2`](https://github.com/tufanbarisyildirim/gonginx/commit/eb064c2fc7592fbac9d408e7c584d70dc577edd9) Add parse options - [`81a37ad`](https://github.com/tufanbarisyildirim/gonginx/commit/81a37ad114f0bee6c7839b9e718114013d9ebcb4) Parse includes recursively - [`8fcd03a`](https://github.com/tufanbarisyildirim/gonginx/commit/8fcd03a8175b8546453ef623b67401b062d0ca6b) Find and Get directives from includes - [`c45fe25`](https://github.com/tufanbarisyildirim/gonginx/commit/c45fe25ce034a9bfd78b600ec4d2ab043c939954) Include parsing tests - [`06a0a4e`](https://github.com/tufanbarisyildirim/gonginx/commit/06a0a4e6c37ae77f2bd595fe9d72a867264dd039) Update testdata - [`91c76bb`](https://github.com/tufanbarisyildirim/gonginx/commit/91c76bbec11d5fdbb616fd675fe40f6c8802b199) Add dumper for include - [`b7a7740`](https://github.com/tufanbarisyildirim/gonginx/commit/b7a7740a4720696d7b61a03bd02d609b8c5cec21) Paser include guard - [`f3fcffe`](https://github.com/tufanbarisyildirim/gonginx/commit/f3fcffe48fc410f299794e71060801a018a2a205) update deps - [`bbe2c0b`](https://github.com/tufanbarisyildirim/gonginx/commit/bbe2c0be907e54f4b44c5f3daacbd0cb17ef8f85) check err first ### 📊 Changes **17 files changed** (+247 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `block.go` (+5 -0) 📝 `dumper.go` (+49 -0) 📝 `go.mod` (+5 -1) 📝 `go.sum` (+26 -0) 📝 `http.go` (+5 -0) 📝 `include.go` (+18 -0) 📝 `parser/parser.go` (+97 -6) 📝 `parser/parser_test.go` (+9 -0) ➕ `testdata/include-glob/conf.d/location-letsencrypt.conf` (+4 -0) ➕ `testdata/include-glob/conf.d/location-root.conf` (+3 -0) ➕ `testdata/include-glob/events.conf` (+3 -0) ➕ `testdata/include-glob/http.conf` (+2 -0) ➕ `testdata/include-glob/nginx.conf` (+5 -0) ➕ `testdata/include-glob/sites-enabled/conf.d/location-letsencrypt.conf` (+2 -0) ➕ `testdata/include-glob/sites-enabled/conf.d/location-root.conf` (+2 -0) ➕ `testdata/include-glob/sites-enabled/example.com.conf` (+6 -0) ➕ `testdata/include-glob/sites-enabled/mysite.com.conf` (+6 -0) </details> ### 📄 Description ## Parse included files recursively Parses the include directives. And lookup directive in the included configs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>