Fix issue with not closed file descriptor

parser/parser.go

@@ -31,6 +31,7 @@ type Parser struct {
 	blockWrappers     map[string]func(*gonginx.Directive) gonginx.IDirective
 	directiveWrappers map[string]func(*gonginx.Directive) gonginx.IDirective
 	commentBuffer     []string
+	file              *os.File
 }
 
 // WithSameOptions copy options from another parser
@@ -87,6 +88,7 @@ func NewParser(filePath string, opts ...Option) (*Parser, error) {
 	l := newLexer(bufio.NewReader(f))
 	l.file = filePath
 	p := NewParserFromLexer(l, opts...)
+	p.file = f
 	return p, nil
 }
 
@@ -152,10 +154,12 @@ func (p *Parser) followingTokenIs(t token.Type) bool {
 
 // Parse the gonginx.
 func (p *Parser) Parse() *gonginx.Config {
-	return &gonginx.Config{
+	c := &gonginx.Config{
 		FilePath: p.lexer.file, //TODO: set filepath here,
 		Block:    p.parseBlock(),
 	}
+	_ = p.Close()
+	return c
 }
 
 // ParseBlock parse a block statement
@@ -345,3 +349,11 @@ func (p *Parser) wrapHTTP(directive *gonginx.Directive) *gonginx.HTTP {
 func (p *Parser) parseUpstreamServer(directive *gonginx.Directive) *gonginx.UpstreamServer {
 	return gonginx.NewUpstreamServer(directive)
 }
+
+// Close closes the file handler and releases the resources
+func (p *Parser) Close() (err error) {
+	if p.file != nil {
+		err = p.file.Close()
+	}
+	return err
+}

parser/parser_test.go

@@ -114,8 +114,9 @@ func TestParser_ParseUpstream(t *testing.T) {
 
 func TestParser_ParseFromFile(t *testing.T) {
 	t.Parallel()
-	_, err := NewParser("../full-example/nginx.conf")
+	p, err := NewParser("../full-example/nginx.conf")
 	assert.NilError(t, err)
+	assert.Assert(t, p.file != nil, "file must be non-nil")
 	_, err2 := NewParser("../full-example/nginx.conf-not-found")
 	assert.ErrorContains(t, err2, "no such file or directory")
 }