Skip to content

fix: added missing methods = "POST" in V2HttpHandlerService inside the Rest Module#1166

Merged
derklaro merged 1 commit intoCloudNetService:nightlyfrom
0xIO32:nightly
Mar 18, 2023
Merged

fix: added missing methods = "POST" in V2HttpHandlerService inside the Rest Module#1166
derklaro merged 1 commit intoCloudNetService:nightlyfrom
0xIO32:nightly

Conversation

@0xIO32
Copy link
Contributor

@0xIO32 0xIO32 commented Mar 17, 2023

Motivation

I wanted to use the api for example create a service from a task but noticed that it tries to return information of a service that does not exists.

Modification

I marked the handleCreateRequest method that it takes only http POST requests rather than GET requests to prevent overlapping with other http handlers.

Result

Now you can create services from a task with the HTTP route again.

@derklaro derklaro added v: 4.X This pull should be included in the 4.0 release t: fix A pull request introducing a fix for a bug. in: module An issue/pull request releated to one of the internal modules labels Mar 18, 2023
@derklaro derklaro added this to the 4.0.0-RC9 milestone Mar 18, 2023
@derklaro derklaro merged commit ffe6d97 into CloudNetService:nightly Mar 18, 2023
derklaro pushed a commit that referenced this pull request Apr 17, 2023
…ather than GET (#1166)

### Motivation
I wanted to use the api for example create a service from a task but
noticed that it tries to return information of a service that does not
exists.

### Modification
I marked the handleCreateRequest method that it takes only http POST
requests rather than GET requests to prevent overlapping with other http
handlers.

### Result
Now you can create services from a task with the HTTP route again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: module An issue/pull request releated to one of the internal modules t: fix A pull request introducing a fix for a bug. v: 4.X This pull should be included in the 4.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants