Faraday: 바디와 함께 삭제

귀하의 검토를 위해 PR이 곧 올라올 것입니다.

안녕하세요 @Carlbc18 입니다 .

위의 변경 사항에 대한 링크를 제공해 주시겠습니까?
HTTP 2.0 사양에만 해당되는 것이므로 이 기능을 신중하게 고려해야 할 수도 있습니다.

https://tools.ietf.org/html/rfc7231#section -4.3.5

참고: 이 동일한 텍스트 블록은 GET && HEAD 요청에도 적용됩니다.

특히 사양의 이 부분:

DELETE 요청 메시지 내의 페이로드에는 정의된 의미가 없습니다.
DELETE 요청에 페이로드 본문을 보내면 기존
요청을 거부하는 구현.

작동하는 테스트로 코드를 변경했습니다(원하는지 확실하지 않음). 이 프로젝트에 대한 홍보를 어떻게 생성합니까?

HTTP 2.0 사양에만 해당되는 것이므로 이 기능을 신중하게 고려해야 할 수도 있습니다.

이것은 실제로 HTTP 1.1의 일부입니다.

PR을 만들고 싶다면 꼭 고려해 보겠습니다.
다른 저장소와 마찬가지로 faraday 포크하고 포크에 변경 사항을 적용한 다음 마스터 브랜치에 대한 PR을 생성할 수 있습니다.

@iMacTia 감사합니다. 내가 올려줄게. 버전 변경에 대한 귀하의 견해는 마침내 이것으로 주요 버전 번호를 갖게 되었습니까? 이 변경 사항은 많은 사람들에게 영향을 줄 수 있습니다. 저는 semver 패턴을 따르고 싶습니다. 이 시점에서 패러데이 버전은 1.0.0이 됩니다. 네 생각을 말해봐.

우리는 확실히 semver를 고수하려고 노력하고 있으며 몇 가지 주요 변경 사항이 있는 v1.0을 예약했습니다.
이 변경으로 인해 이전 버전과의 호환성이 중단될 경우 버전 1.0(현재 고정된 릴리스 날짜가 없음)에서만 릴리스됩니다.
그러나 내 이해는 이제 선택적으로 GET, HEAD 및 DELETE 요청에 본문을 추가할 수 있어야 하므로 이전 버전과 호환되어서는 안 된다는 것입니다. 내가 뭔가를 놓치고 있습니까?

@iMacTia i 버전이 0.13.0이고 PR이 올라왔습니다. 의견이 있을 것이라고 확신하므로 이에 대한 다른 대화를 PR에 전달할 수 있습니다. 다시 한번 감사합니다!

이것은 내 https://github.com/lostisland/faraday/pull/695#issuecomment -305145499에 대한 후속 조치입니다.
우선 우리는 이것이 어떻게 작동하기를 원하는지 결정해야 합니다.
변경 사항이 이전 버전과 호환되어야 한다고 가정하면 아래는 여전히 작동합니다.

conn = Faraday.new(...)
conn.get('/path', {page: 1, per: 10})

# performs "GET /path?page=1&per=10"

내 의견에 명시된 옵션 1은 이 동작을 구성 가능하게 만드는 것입니다.

conn = Faraday.new(..., standard: :http1_1)
conn.get('/path', {page: 1, per: 10})

# performs "GET /path" with body "page=1&per=10" (assuming www_form_url_encoded request)

대신 옵션 2는 더 강력한 것을 의미합니다.

conn = Faraday.new(...) # note no special config here
conn.get('/path', {page: 1, per: 10}, body: {some_key: 'some_value')

# performs "GET /path?page=1&per=10" with body "some_key=some_value"

하지만 문제가 있습니다. ruby ​​1.9는 키워드 매개변수를 지원하지 않으므로 옵션 2는 ruby ​​>= 2에서만 작동할 수 있습니다. 😞

루비 1.9에서 작동하지 않는 것과 관련하여 괜찮다고 생각합니다. Ruby 1.9는 오래되었습니다. 나는 우리가 패러데이의 v1.0 릴리스와 인라인이 될 최소 루비 버전을 위해 적어도 2.0으로 버전을 올릴 것을 지지합니다. 또한 옵션 2는 parms 및/또는 선택적 본문을 전달할 수 있는 옵션이 있으므로 http 1.1 사양을 더 밀접하게 따릅니다.

Ruby 1.9에서 작동하는 http 사양 기능과 하위 호환성을 유지하려고 하면 문제가 발생할 것 같습니다. 이 기능은 V1 of Faraday에 더 적합하지 않을까 생각합니다. 이 변경 사항(ruby v, params 전달)을 지원하기 위해 너무 많은 변수가 필요한 것 같습니다. 내가 과한 생각을 하고 있는 건 아닐까....

루비 1.9에서 작동하지 않는 것과 관련하여 괜찮다고 생각합니다. Ruby 1.9는 오래되었습니다. 나는 우리가 패러데이의 v1.0 릴리스와 인라인이 될 최소 루비 버전을 위해 적어도 2.0으로 버전을 올릴 것을 지지합니다. 또한 옵션 2는 parms 및/또는 선택적 본문을 전달할 수 있는 옵션이 있으므로 http 1.1 사양을 더 밀접하게 따릅니다.

위의 모든 내용에 전적으로 동의합니다. 이것이 우리가 여전히 v1.0을 기다려야 함을 의미하지만 그것이 실제로 최선의 조치입니다.

따라서 v1.0 이전 버전과 호환되는 API를 정의하기 위해 다음을 시도할 수 있습니다.

GET, HEAD, 삭제 요청

conn = Faraday.new(...) # note no special config here
conn.get(path, url_params, body: {...})
# second parameter is URL PARAMS, optionally accept a named parameter for request body
# works the same for head and delete

POST, PUT, 패치 요청

conn = Faraday.new(...) # note no special config here
conn.post(path, body, url_params: {...})
# second parameter is REQUEST BODY, optionally accept a named parameter for url_params
# works the same for put and patch

이런 식으로 가능한 모든 조합을 다루고 post/put/patch 요청도 확장한다고 생각합니다!

확실히 동의합니다. 내 PR을 거부하고 이를 위해 새 PR을 열겠습니까? 작업할 수 있는 V1 분기 컷이 있습니까?

그것을 닫고 완전히 다를 것이기 때문에 처음부터 시작하게 되어 기쁩니다. 그 죄송합니다!

병합하기 전에 V1 브랜치를 생성할 것이므로 마스터에서 자유롭게 시작하면 PR을 리디렉션할 것입니다. :)

안녕하세요 @Carlbc18 님이 이제 1.0 지점을 사용할 수 있음을 알려드리고자 합니다 😃.

고마워, 고마워! 😀

아직 이것을 기다리고 있는 사람을 위해 제 경우에 효과가 있는 것으로 보이는 다음 해결 방법을 찾았습니다.

conn = Faraday.new
body = {
some_key: 'some value'
}
conn.run_request(:delete, url, body, headers)

삭제가 더 이상 기본적으로 본문을 nils하지 않을 때까지 이것이 누군가를 돕기를 바랍니다.

그렇다면 언제 사용할 수 있는지에 대한 업데이트가 있습니까?

아직 PR을 본 적은 없지만 v1에 대한 작업이 천천히 진행되고 있습니다.
직접 작업할 수는 있지만 현재로서는 견적을 드릴 수 없습니다.
요청이 어떻게 작동해야 하는지에 대한 업데이트된 API는 위의 내 의견에 설명되어 있으므로 누구든지 나보다 먼저 시도하고 싶다면 PR을 환영합니다.

이전 버전과 완전히 호환되는 솔루션의 경우 PR을 마스터에서 분기할 수 있으며 다른 0.x 릴리스를 만들 수 있습니다.
변경 사항으로 인해 이전 버전과의 호환성이 중단될 경우 1.0 분기에서 분기하는 것이 좋습니다.

https://github.com/lostisland/faraday/pull/855 는 이를 지원하기 위한 PR 초안입니다.

이 기능을 거부합니다. 패러데이에서 블록 형식을 사용하여 본문으로 요청할 수 있습니다.

conn.delete(url) do |req|
  req.body = { some_key: 'some value' }
end

이 아이디어는 본체를 단일 라이너로 간단히 삭제했지만 구현은 내가 원하는 것보다 더 복잡하고 놀라운 동작을 추가합니다. 이 블록 형식은 이미 패러데이에서 잘 작동합니다. 메서드에 더 많은 인수를 추가하는 것보다 클래스의 속성을 수정하는 것을 선호합니다.

conn.delete('/foo/1') do |req|
  req.headers["X-Men"] = "marvel"
  req.params[:confirm] = 1
  req.body = { some_key: 'some value' }
end