Skip to content

Prioritise API with api write=on #1315

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

Merged
merged 17 commits into from
Nov 4, 2025
Merged

Prioritise API with api write=on #1315

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
d11d7a5
selecting API with write access
Akshay2191 Sep 30, 2025
99d0db9
parsing the plusAPIs
Akshay2191 Oct 1, 2025
14977b6
reducing Cognitive Complexity
Akshay2191 Oct 1, 2025
5552230
fixed integration test, reduced complexity
Akshay2191 Oct 3, 2025
ed28bf1
fixing integration test
Akshay2191 Oct 3, 2025
6a55e69
fixing integration test
Akshay2191 Oct 3, 2025
66cc63b
fixing integration test
Akshay2191 Oct 3, 2025
aeb6c68
sorting while creating context
Akshay2191 Oct 6, 2025
a5e6be4
reduced Lint cognitive complexity
Akshay2191 Oct 7, 2025
c5bbfec
worked on PR comments
Akshay2191 Oct 8, 2025
a740b69
fixed lint error
Akshay2191 Oct 8, 2025
a38d37f
Modified log level to info
Akshay2191 Oct 14, 2025
46aaa89
removing the write-enable from MPI
Akshay2191 Oct 15, 2025
af70f5d
fix pipeline
aphralG Oct 16, 2025
32f010c
updated traverseAPI to read all APIs
Akshay2191 Oct 22, 2025
a69616b
reverted the fix done for git pipeline issue
Akshay2191 Nov 3, 2025
2a449d0
sorting after getting all apis
Akshay2191 Nov 3, 2025
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
Modified log level to info
  • Loading branch information
@Akshay2191
Akshay2191 committed Oct 14, 2025
commit a38d37fe1813affb27680b9a0efe2b624dda9c73
2 changes: 1 addition & 1 deletion internal/datasource/config/nginx_config_parser.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -938,7 +938,7 @@ func (ncp *NginxConfigParser) sortPlusAPIs(ctx context.Context, apis []*model.AP
}

if !foundWriteEnabled && len(apis) > 0 {
slog.WarnContext(ctx, "No write-enabled NGINX Plus API found. Defaulting to read-only API")
slog.InfoContext(ctx, "No write-enabled NGINX Plus API found. Defaulting to read-only API")
return apis
}

Expand Down
Loading