fix: break resource create/update handlers on error (closes #1464)
This commit is contained in:
		
							parent
							
								
									6b19ab6613
								
							
						
					
					
						commit
						5072bbb2cb
					
				| 
						 | 
				
			
			@ -4,7 +4,6 @@ import (
 | 
			
		|||
	"context"
 | 
			
		||||
	"fmt"
 | 
			
		||||
	"io"
 | 
			
		||||
	"io/ioutil"
 | 
			
		||||
	"net/http"
 | 
			
		||||
	"net/url"
 | 
			
		||||
	"os"
 | 
			
		||||
| 
						 | 
				
			
			@ -96,10 +95,6 @@ func resourcePostHandler(fileCache FileCache) handleFunc {
 | 
			
		|||
			return http.StatusForbidden, nil
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		defer func() {
 | 
			
		||||
			_, _ = io.Copy(ioutil.Discard, r.Body)
 | 
			
		||||
		}()
 | 
			
		||||
 | 
			
		||||
		// Directories creation on POST.
 | 
			
		||||
		if strings.HasSuffix(r.URL.Path, "/") {
 | 
			
		||||
			err := d.user.Fs.MkdirAll(r.URL.Path, 0775)
 | 
			
		||||
| 
						 | 
				
			
			@ -154,11 +149,6 @@ var resourcePutHandler = withUser(func(w http.ResponseWriter, r *http.Request, d
 | 
			
		|||
		return http.StatusForbidden, nil
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	defer func() {
 | 
			
		||||
		_, _ = io.Copy(ioutil.Discard, r.Body)
 | 
			
		||||
	}()
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
	// Only allow PUT for files.
 | 
			
		||||
	if strings.HasSuffix(r.URL.Path, "/") {
 | 
			
		||||
		return http.StatusMethodNotAllowed, nil
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue