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 issue with getFirmwareConfig (400: not found) #6655

Closed
2 tasks done
clementmoulin opened this issue Jul 7, 2023 · 6 comments
Closed
2 tasks done

API issue with getFirmwareConfig (400: not found) #6655

clementmoulin opened this issue Jul 7, 2023 · 6 comments

Comments

@clementmoulin
Copy link
Contributor

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

Since last versions (I would say 23.1.x cannot tell exactly which one), the route API core/firmware/getfirmwareconfig is not working anymore.

To Reproduce

curl -k -u '...':'...' http://localhost/api/core/firmware/getfirmwareconfig -v
[...]
> GET /api/core/firmware/getfirmwareconfig HTTP/1.1
> Host: localhost
[...]
< HTTP/1.1 400 Bad Request
[...]
< Server: OPNsense
<
* Connection #0 to host localhost left intact
{"message":"action getfirmwareconfig not found","status":400}

Expected behavior

Answer from API :)

Describe alternatives you considered

N/A

Screenshots

N/A

Relevant log files

N/A

Additional context

N/A

Environment

Software version used and hardware type if relevant, e.g.:

OPNsense 23.1.5 to 23.1.11 (amd64, OpenSSL).

@clementmoulin
Copy link
Contributor Author

OK. I have made some investigations through GUI, and it seems that routes have been updated and now are named api/firmware/set, api/firmware/get and api/firmware/get_options.

Documentation should be updated : https://docs.opnsense.org/development/api/core/firmware.html

@AdSchellevis
Copy link
Member

uniformed in 4c65524, I'll make sure to update the api docs before the 23.7 release

@AdSchellevis
Copy link
Member

I think we can close this with opnsense/docs@8f5ea36

@clementmoulin
Copy link
Contributor Author

clementmoulin commented Jul 7, 2023

I don't see any update about core/firmware/ routes in that commit.

@AdSchellevis AdSchellevis reopened this Jul 7, 2023
@AdSchellevis
Copy link
Member

it looks like this one is excluded from our automated procedures, let me check if we can fix this for future changes as well.

@AdSchellevis
Copy link
Member

we do have to do this manually it seems, opnsense/docs@e747ad1 should be it then.

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

No branches or pull requests

2 participants