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.

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

Jump to bottom

proposal: x/net/webdav: make webdav/internal/xml module public #53527

Closed
xepozz opened this issue Jun 24, 2022 · 5 comments
Closed

proposal: x/net/webdav: make webdav/internal/xml module public #53527

xepozz opened this issue Jun 24, 2022 · 5 comments
Labels
FrozenDueToAge Proposal
Milestone
Proposal

Comments

@xepozz
Copy link

xepozz commented Jun 24, 2022

I found the webdav package when I had been doing custom WebDAV server.
It looks good if you want to just serve a directory. But when you are developing WebDAV server-proxy that just calls APIs it's partially useless because you need to implement each WebDAV http method by myself. Need to implement methods, but not a response factory.

I see that the package uses internal module xml to create a response with right structure. It would be nice to open it for developers or split into a separated package. I would make a PR but I'm not a gopher, just a fan.

Glad to get thoughts about the idea.
@gopherbot gopherbot added this to the Unreleased milestone Jun 24, 2022
@ianlancetaylor ianlancetaylor changed the title x/net/webdav: proposal: Make webdav/internal/xml module public proposal: x/net/webdav: make webdav/internal/xml module public Jun 29, 2022
@ianlancetaylor ianlancetaylor modified the milestones: Unreleased, Proposal Jun 29, 2022
@ianlancetaylor
Copy link
Member

CC @nigeltao

@rsc
Copy link
Contributor

rsc commented Jun 29, 2022

If it's useful, feel free to copy it (subject to the license requirements of course). It seems unlikely we'll want to commit to more webdav-related API.

@rsc
Copy link
Contributor

rsc commented Jul 1, 2022

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

@rsc
Copy link
Contributor

rsc commented Jul 13, 2022

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

@rsc
Copy link
Contributor

rsc commented Jul 20, 2022

No change in consensus, so declined.
— rsc for the proposal review group

@rsc rsc closed this as completed Jul 20, 2022
@rsc rsc moved this to Declined in Proposals Aug 10, 2022
@rsc rsc added this to Proposals Aug 10, 2022
@golang golang locked and limited conversation to collaborators Jul 20, 2023
@rsc rsc removed this from Proposals Jul 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Proposal
Projects
None yet
Milestone
Proposal
Development

No branches or pull requests
4 participants
@rsc @ianlancetaylor @xepozz @gopherbot