Merge pull request #10854 from adlius/fix-citation-addons

[ENG-6731] Fix zotero connection
CenterForOpenScience · Dec 17, 2024 · b2f1a2b · b2f1a2b
2 parents ebbb59c + abccfc5
commit b2f1a2b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/addons/base/tests/views.py b/addons/base/tests/views.py
@@ -82,6 +82,8 @@ def test_oauth_finish_enable_gv(self, mock_requests_get):
         query_params = {
             'code': 'somecode',
             'state': 'somestatetoken',
+            'oauth_token': 'someoauthtoken',
+            'oauth_verifier': 'someverifier',
         }
         with override_flag(ENABLE_GV, active=True):
             request_url = urlunparse(urlparse(url)._replace(query=urlencode(query_params)))

diff --git a/osf/external/gravy_valet/translations.py b/osf/external/gravy_valet/translations.py
@@ -38,7 +38,7 @@ class _LegacyConfigsForWBKey(enum.Enum):
     onedrive = OneDriveAddonAppConfig
     owncloud = OwnCloudAddonAppConfig
     s3 = S3AddonAppConfig
-    zotero = ZoteroAddonAppConfig
+    zotero_org = ZoteroAddonAppConfig
     mendeley = MendeleyAddonConfig
 
 

diff --git a/website/oauth/views.py b/website/oauth/views.py
@@ -73,9 +73,14 @@ def oauth_callback(service_name, auth):
 def _forward_to_addon_service(oauth_version):
     code = request.args.get('code')
     state = request.args.get('state')
+    oauth_token = request.args.get('oauth_token')
+    oauth_verifier = request.args.get('oauth_verifier')
     query_params = {
         'code': code,
         'state': state,
+        'oauth_token': oauth_token,
+        'oauth_verifier': oauth_verifier,
     }
+
     gv_url = urlunparse(urlparse(GRAVYVALET_URL)._replace(path=f'/v1/{oauth_version}/callback', query=urlencode(query_params)))
-    requests.get(gv_url)
+    requests.get(gv_url, cookies=request.cookies)