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.

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

API Improvements #4356

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

Sander0542
Copy link
Contributor

@Sander0542 Sander0542 commented Feb 7, 2025

Streams List

  • Currently, the stream list contains the proxy host object. This needs to be replaced with the stream object.

Access List

  • The access list API is missing definitions for the clients and items arrays.
  • The access list API says is supports proxy hosts expansions, but the backend does not allow this.
    .allowGraph('[owner,items,clients]')
  • Moved duplicate properties to common

Expansion query parameter

  • In the API schema, only the list endpoints support the expand query parameter, but query parameters are also supported in the get endpoints.

User & Permissions

  • The user object did not contain the permissions property
  • The permissions object did not container required fields

@Sander0542 Sander0542 changed the title Return stream object in stream list API Improvements Feb 8, 2025
@nginxproxymanagerci
Copy link

Docker Image for build 7 is available on
DockerHub
as nginxproxymanager/nginx-proxy-manager-dev:pr-4356

Note: ensure you backup your NPM instance before testing this image! Especially if there are database changes
Note: this is a different docker image namespace than the official image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant