get rid of public link url hash

changelog/unreleased/change-public-link-hash.md

@@ -0,0 +1,7 @@
+Change: Remove hash from public link urls
+
+Public link urls do not contain the hash anymore, this is needed to support the ocis and web history mode.
+
+https://github.com/cs3org/reva/pull/2596
+https://github.com/owncloud/ocis/pull/3109
+https://github.com/owncloud/web/pull/6363

internal/http/services/owncloud/ocdav/ocdav.go

@@ -229,8 +229,9 @@ func (s *svc) Handler() http.Handler {
 			head, r.URL.Path = router.ShiftPath(r.URL.Path)
 			if head == "s" {
 				token := r.URL.Path
-				url := s.c.PublicURL + path.Join("#", head, token)
-				http.Redirect(w, r, url, http.StatusMovedPermanently)
+				rURL := s.c.PublicURL + path.Join(head, token)
+
+				http.Redirect(w, r, rURL, http.StatusMovedPermanently)
 				return
 			}
 		}

internal/http/services/owncloud/ocs/conversions/main.go

@@ -23,6 +23,7 @@ import (
 	"context"
 	"fmt"
 	"net/http"
+	"net/url"
 	"path"
 	"time"
 
@@ -205,14 +206,17 @@ func CS3Share2ShareData(ctx context.Context, share *collaboration.Share) (*Share
 
 // PublicShare2ShareData converts a cs3api public share into shareData data model
 func PublicShare2ShareData(share *link.PublicShare, r *http.Request, publicURL string) *ShareData {
+	sdURL, _ := url.ParseRequestURI(publicURL)
+	sdURL.Path = path.Join(sdURL.Path, "s", share.Token)
+
 	sd := &ShareData{
 		// share.permissions are mapped below
 		// Displaynames are added later
 		ShareType:    ShareTypePublicLink,
 		Token:        share.Token,
 		Name:         share.DisplayName,
 		MailSend:     0,
-		URL:          publicURL + path.Join("/", "#/s/"+share.Token),
+		URL:          sdURL.String(),
 		UIDOwner:     LocalUserIDToString(share.Creator),
 		UIDFileOwner: LocalUserIDToString(share.Owner),
 	}