Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connect BioModels/All Curated ModelIds fails: Exception: (307, u'Temporary Redirect') #230

Open
yurivict opened this issue Dec 18, 2017 · 8 comments

Comments

@yurivict
Copy link

The progress indicator goes up to 13% and then throws this exception:

Traceback (most recent call last):
  File "/usr/local/lib/moose/gui/biomodelsclient.py", line 240, in runQuery
    name = unicode(self.client.service.getModelNameById(item)).encode("utf-8")
  File "/usr/local/lib/moose/gui/suds/client.py", line 521, in __call__
    return client.invoke(args, kwargs)
  File "/usr/local/lib/moose/gui/suds/client.py", line 581, in invoke
    result = self.send(soapenv)
  File "/usr/local/lib/moose/gui/suds/client.py", line 619, in send
    description=tostr(e), original_soapenv=original_soapenv)
  File "/usr/local/lib/moose/gui/suds/client.py", line 677, in process_reply
    raise Exception((status, description))
Exception: (307, u'Temporary Redirect')
@dilawar
Copy link
Contributor

dilawar commented Dec 28, 2017

AFAIK, suds is not being maintained anymore. MOOSE uses suds to download the model. @hrani (and probably @subhacom ) might be able to comment on if there is plan to continue with suds to drop it altogether in favor of 'curl' or urllib2 etc.

@subhacom
Copy link
Collaborator

curl or urllib2 is not the same thing as suds, which is a SOAP client. There is a fork of suds which has been maintained irregularly, and if anybody is willing to port the code, zeep might be a better alternative at present.
However I wonder how many users are out there who might want this feature.

@dilawar
Copy link
Contributor

dilawar commented Dec 29, 2017

Well, IMO either a feature should be maintained or removed. Let @hrani and @upibhalla take a call on this one. zeep looks promising; it depends only on python-lxml which is a well-maintained library and available on almost all linux flavors for which we build pacakges.

@dilawar dilawar assigned upibhalla and unassigned dilawar Dec 29, 2017
@dilawar dilawar added the task label Dec 29, 2017
@hrani
Copy link
Contributor

hrani commented Dec 29, 2017 via email

@dilawar
Copy link
Contributor

dilawar commented Mar 6, 2018

@hrani Is it planned for chennapoda?

@hrani
Copy link
Contributor

hrani commented Mar 6, 2018 via email

@dilawar
Copy link
Contributor

dilawar commented Jul 31, 2018

@hrani Is they any update on this ticket?

@dilawar
Copy link
Contributor

dilawar commented Mar 10, 2020

Pinging if there is update here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants