http: handle all of the http.Method values in .str() (#6002)

pull/5940/head
Carlos Esquerdo Bernat 2020-07-28 09:39:10 +02:00 committed by GitHub
parent 8b66816bdc
commit 28657fe7cb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 48 additions and 35 deletions

View File

@ -14,19 +14,6 @@ const (
bufsize = 1536
)
pub enum Method {
get
post
put
header
delete
options
head
trace
connect
patch
}
pub struct Request {
pub mut:
method Method
@ -84,28 +71,6 @@ fn (methods []Method) contains(m Method) bool {
return false
}
fn (m Method) str() string {
return match m {
.get { 'GET' }
.post { 'POST' }
.header { 'HEADER' }
.put { 'PUT' }
.connect { 'CONNECT' }
else { '' }
}
}
pub fn method_from_str(m string) Method {
return match m {
'GET' { Method.get }
'POST' { Method.post }
'HEADER' { Method.header }
'PUT' { Method.put }
'CONNECT' { Method.connect }
else { Method.get } // should we default to GET?
}
}
pub fn get(url string) ?Response {
return fetch_with_method(.get, url, FetchConfig{})
}

View File

@ -0,0 +1,48 @@
// Copyright (c) 2019-2020 Alexander Medvednikov. All rights reserved.
// Use of this source code is governed by an MIT license
// that can be found in the LICENSE file.
module http
// The methods listed here are some of the most used ones, ordered by
// commonality. A comprehensive list is available at:
// https://www.iana.org/assignments/http-methods/http-methods.xhtml
pub enum Method {
get
post
put
head
delete
options
trace
connect
patch
}
fn (m Method) str() string {
return match m {
.get { 'GET' }
.post { 'POST' }
.put { 'PUT' }
.head { 'HEAD' }
.delete { 'DELETE' }
.options { 'OPTIONS' }
.trace { 'TRACE' }
.connect { 'CONNECT' }
.patch { 'PATCH' }
}
}
pub fn method_from_str(m string) Method {
return match m {
'GET' { Method.get }
'POST' { Method.post }
'PUT' { Method.put }
'HEAD' { Method.head }
'DELETE' { Method.delete }
'OPTIONS' { Method.options }
'TRACE' { Method.trace }
'CONNECT' { Method.connect }
'PATCH' { Method.patch }
else { Method.get } // should we default to GET?
}
}