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

Incorrect HTTP method for doc/ls #97

Closed
IgorShadurin opened this issue Aug 11, 2021 · 0 comments
Closed

Incorrect HTTP method for doc/ls #97

IgorShadurin opened this issue Aug 11, 2021 · 0 comments
Assignees
Labels
bug Something isn't working P1 Priority 1

Comments

@IgorShadurin
Copy link
Collaborator

I seems like HTTP parser expects POST method at https://github.com/fairDataSociety/fairOS-dfs/blob/master/pkg/api/doc_ls.go, but at

docRouter.HandleFunc("/ls", handler.DocListHandler).Methods("GET")
it defined as GET.

Solution: fix HTTP parse at doc_ls.go

@IgorShadurin IgorShadurin added the bug Something isn't working label Aug 11, 2021
@crtahlin crtahlin added the P1 Priority 1 label Aug 12, 2021
@asabya asabya self-assigned this Nov 11, 2021
asabya added a commit that referenced this issue Nov 11, 2021
@asabya asabya closed this as completed Nov 18, 2021
asabya added a commit that referenced this issue Dec 13, 2021
* addressing #119, uploading same file multiple times

* addressing #122, file delete Matadata unmarshal

* addressing #123, fix response for non-existing pod

* addressing #127, server requires postageBlockId to run

* addressing #116, non-existing dir response

* addressing #119, backup old file in case same name

* code cleanup

* code cleanup

* code cleanup

* code cleanup

* addressing #117 and #118, issues related to pod deletion (#1)

* addressing #117 and #118, issues related to pod deletion

* fix reploading deleted file

* fix #132, #133

* fixes #141, rmdir updates feed with magicword

* fixes #134

* fixes #135 & #136

* fixing #139

* fix #140, accept mutable bool in request

* minor typo

* #97 handle query param for doc-ls

* #102 handle exception for invalid json

* #106 handled panic

* related to #100, doc new podname missing

* fix #124, handle removed root directory

* handle #96

* handle #24, use shlex to parse prompt args

* fixed #19, pod deletion requires password

* check for root dir presence in mkdir

* mod tidy (#146)

* mod tidy

* manually restore vendor

* manually restore vendor

* upload err fix

* fix #86, shared pod removed from list

* minor changes

* fix #147, check read only feed before access time update in download

* fix #148, close response body in bee client

* minor file deletion test added

* #92, table name added in count response

* fix w/ deepsource recommendation

* fix w/ deepsource recommendation

* #102, add test for invalid json

* 404 on file not present

* add get endpoint for download

* fix #158

* resolve anti-pattern

* bump version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1 Priority 1
Projects
None yet
Development

No branches or pull requests

3 participants