[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/graphql-mode f3cfd9fc1e 062/122: Don't send the query as a
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/graphql-mode f3cfd9fc1e 062/122: Don't send the query as a query parameter |
Date: |
Sat, 29 Jan 2022 08:03:33 -0500 (EST) |
branch: elpa/graphql-mode
commit f3cfd9fc1e458ecc37fc90738e5bdaf084d5d433
Author: David Vázquez Púa <davazp@gmail.com>
Commit: David Vázquez Púa <davazp@gmail.com>
Don't send the query as a query parameter
The query is sent in the post body, so shouldn't be included
additionally in the query parameter.
---
graphql-mode.el | 27 +++++++++++++--------------
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/graphql-mode.el b/graphql-mode.el
index 67d59b2b45..9d57d0918c 100644
--- a/graphql-mode.el
+++ b/graphql-mode.el
@@ -87,8 +87,9 @@ mutation or subscription). OPERATION is a name for the
operation. VARIABLES is the JSON string that specifies the values
of the variables used in the query."
(let* ((body (graphql-encode-json query operation variables))
- (url (format "%s?query=%s" graphql-url (url-encode-url body))))
- (with-temp-buffer (graphql-post-request url query operation variables))))
+ (url graphql-url))
+ (with-temp-buffer
+ (graphql-post-request url query operation variables))))
(defun graphql-post-request (url query &optional operation variables)
"Make post request to graphql server with url and body.
@@ -98,18 +99,16 @@ QUERY query definition(s) of query, mutation, and/or
subscription
OPERATION name of the operation if multiple definition is given in QUERY
VARIABLES list of variables for query operation"
(let* ((body (graphql-encode-json query operation variables))
- (endpoint (car (split-string url "?")))
- (response (request
- url
- :type "POST"
- :data body
- :headers '(("Content-Type" . "application/json"))
- :parser 'json-read
- :sync t
- :complete (lambda (&rest _)
- (message "%s" (if (string-equal "" operation)
- endpoint
- (format "%s?operationName=%s"
endpoint operation)))))))
+ (response (request url
+ :type "POST"
+ :data body
+ :headers '(("Content-Type" . "application/json"))
+ :parser 'json-read
+ :sync t
+ :complete (lambda (&rest _)
+ (message "%s" (if (string-equal ""
operation)
+ url
+ (format
"%s?operationName=%s" endpoint operation)))))))
(json-encode (request-response-data response))))
(defun graphql-beginning-of-query ()
- [nongnu] elpa/graphql-mode 3965a4c370 046/122: add request dependency, (continued)
- [nongnu] elpa/graphql-mode 3965a4c370 046/122: add request dependency, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 52137014df 073/122: Update packages, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode c3471642f5 072/122: Merge pull request #19 from cbowdon/master, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 880701685d 038/122: Add some types/field to the test schema, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 0f2b4b1604 077/122: Merge branch 'next', ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode d0a85fe1d3 067/122: Shorten long lines, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 7aa3532aed 026/122: Don't depend on json-mode, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 0b41444096 028/122: Add installation notes, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 43434aa6d5 061/122: Format some docs and docstrings, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 8b2fe76635 057/122: fix a recursive bug in graphql-current-operation, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode f3cfd9fc1e 062/122: Don't send the query as a query parameter,
ELPA Syncer <=
- [nongnu] elpa/graphql-mode 301d2419a0 012/122: Prompt the user for the GraphQL, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 17c2150ff6 058/122: rebase branch 'queries support' into branch 'next', ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 83e3a73f21 033/122: Fix: don't add slash between url and query parameters, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 22a4340ec3 044/122: add post body json support: query, operationName, variables, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 41df2a3140 019/122: Add docstring for graphql-mode, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode d5cfe9326c 074/122: Rename graphql-headers to graphql-extra-headers, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 5971b69c8c 076/122: Fix: respect local value of graphql-url when posting a request, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 5e7665dfc4 059/122: forget the actual changes, (I have two location to dev/test), ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 2326c1fb18 034/122: Merge pull request #7 from eoma/fix-url, ELPA Syncer, 2022/01/29
- [nongnu] elpa/graphql-mode 04e56ae626 053/122: Disable json-mode for response temporarily, ELPA Syncer, 2022/01/29