Skip to content

Commit

Permalink
move final few terrain methods to PyCyAPI (#299)
Browse files Browse the repository at this point in the history
  • Loading branch information
wpbonelli committed May 8, 2022
1 parent 2882f98 commit 9db346d
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 87 deletions.
8 changes: 1 addition & 7 deletions docker-compose.dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -247,10 +247,4 @@ services:
interval: 60s
timeout: 60s
retries: 2
start_period: 20s
# TODO: singularity-in-docker and docker-in-docker containers as test agents
# sind:
# image: computationalplantscience/sind
# command: ["/usr/sbin/sshd", "-D"]
# working_dir: /opt/work
# privileged: true
start_period: 20s
4 changes: 2 additions & 2 deletions plantit/plantit/datasets/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
from pycyapi.clients import TerrainClient, AsyncTerrainClient

import plantit.queries as q
import plantit.terrain as terrain
from plantit.datasets.models import DatasetAccessPolicy, DatasetRole
from plantit.miappe.models import Investigation, Study
from plantit.notifications.models import Notification
Expand Down Expand Up @@ -56,7 +55,8 @@ async def shared(request):

policies = await sync_to_async(list)(DatasetAccessPolicy.objects.filter(guest=request.user))
paths = [policy.path for policy in policies]
dirs = await terrain.get_dirs_async(paths, request.user.profile.cyverse_access_token, int(settings.HTTP_TIMEOUT))
client = AsyncTerrainClient(request.user.profile.cyverse_access_token, int(settings.HTTP_TIMEOUT))
dirs = await client.get_dirs_async(paths)
return JsonResponse({'datasets': [dir for dir in dirs]})


Expand Down
12 changes: 9 additions & 3 deletions plantit/plantit/queries.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
from django.utils import timezone

from pycyapi.clients import TerrainClient
import plantit.terrain as terrain
from pycyapi.exceptions import Unauthorized
import plantit.mapbox as mapbox
from plantit import github as github
from plantit import loess as loess
Expand Down Expand Up @@ -526,8 +526,14 @@ def refresh_user_cyverse_tokens(user: User):
return

try:
access_token, refresh_token = terrain.refresh_tokens(username=user.username, refresh_token=user.profile.cyverse_refresh_token)
except terrain.Unauthorized as e:
client = TerrainClient(user.profile.cyverse_access_token)
access_token, refresh_token = client.refresh_tokens(
username=user.username,
client_id=settings.CYVERSE_CLIENT_ID,
client_secret=settings.CYVERSE_CLIENT_SECRET,
refresh_token=user.profile.cyverse_refresh_token,
redirect_uri=settings.CYVERSE_REDIRECT_URL)
except Unauthorized as e:
decoded = jwt.decode(user.profile.cyverse_refresh_token, options={
'verify_signature': False,
'verify_aud': False,
Expand Down
69 changes: 0 additions & 69 deletions plantit/plantit/terrain.py

This file was deleted.

6 changes: 0 additions & 6 deletions plantit/plantit/tests/integration/test_terrain.py

This file was deleted.

0 comments on commit 9db346d

Please sign in to comment.