Opened 7 years ago

Closed 5 years ago

#13154 closed defect (fixed)

Bug on Sage simple server

Reported by: ares Owned by: Ares Ribó
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: notebook Keywords:
Cc: jason Merged in:
Authors: Reviewers: Jason Grout, Karl-Dieter Crisman, Kannappan Sampath
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by kcrisman)

We have the following problems:

nb = test_notebook(passwd, secure=False, address='localhost', port=port, verbose=True) 
bash-3.2$ sage

----------------------------------------------------------------------[[BR]]

| Sage Version 5.0.1, Release Date: 2012-06-10                       |[[BR]]

| Type notebook() for the GUI, and license() for information.        |[[BR]]

----------------------------------------------------------------------[[BR]]

Loading Sage library. Current Mercurial branch is: d6.2 [[BR]]

sage: from sage.server.misc import find_next_available_port [[BR]]

sage: port = find_next_available_port(9000, verbose=False) [[BR]]

sage: from sagenb.notebook.notebook_object import test_notebook [[BR]]

sage: passwd = str(randint(1,1<<128)) [[BR]]

sage: nb = test_notebook(passwd, secure=False, address='localhost', port=port, verbose=True) [[BR]]

--------------------------------------------------------------------------- [[BR]]
 
TypeError                                 Traceback (most recent call last) [[BR]]

/Applications/sage/<ipython console> in <module>()

TypeError: test_notebook() got an unexpected keyword argument 'address'

''This is the first error. It seems that `address` is not currently (in 5.0.1) a valid option.
Must update the docs.
''
[[BR]]
[[BR]]
[[BR]]




sage: nb = test_notebook(passwd, secure=False, port=port, verbose=True) 

Notebook started.

sage: import urllib, re

sage: def get_url(url): h = urllib.urlopen(url); data = h.read(); h.close(); return data

....: 

sage: sleep(1)

sage: login_page = get_url('http://localhost:%s/simple/login?username=admin&password=%s' % (port, 

passwd))

sage: print login_page

{

"session": "fcee333f07d53c13a46f65b9c4a15473"

}

___S_A_G_E___


sage: session = re.match(r'.*"session": "([^"]*)"', login_page, re.DOTALL).groups()[0]

sage: sleep(0.5)

sage: print get_url('http://localhost:%s/simple/compute?session=%s&code=2*2&timeout=60' % (port, 

session))

{

"status": "done",

"files": [],

"cell_id": 2

}

___S_A_G_E___

4

Just repeating this command 3 times, it gets stuck.

Change History (10)

comment:1 Changed 7 years ago by kcrisman

  • Authors Jordi Saludes deleted
  • Cc jason added
  • Description modified (diff)

If I'm not mistaken, the Sage simple server is more or less deprecated. The Sage cell server seems to be along the lines of replacing it. Jason, is that correct?

comment:2 Changed 7 years ago by jason

It does not work with the new notebook, and no one has volunteered to fix it yet. Pull requests are welcome, of course. The Sage Cell Server does do (most of) the things you'd want the simple server to do.

comment:3 Changed 7 years ago by kcrisman

I guess what I'm wondering is whether this ticket should be closed. It sounds like you're saying that this could be fixed, just that no one has volunteered yet.

comment:4 Changed 7 years ago by jason

My best guess is that the current simple server code is dead and will need to be rewritten from scratch to be useful again.

comment:5 Changed 6 years ago by kcrisman

  • Milestone changed from sage-5.11 to sage-duplicate/invalid/wontfix
  • Reviewers set to Jason Grout, Karl-Dieter Crisman
  • Status changed from new to needs_review

comment:6 Changed 6 years ago by kcrisman

  • Status changed from needs_review to positive_review

comment:7 Changed 6 years ago by kcrisman

  • Status changed from positive_review to needs_info

Note that #11409 would remove this completely.

comment:8 Changed 5 years ago by knsam

As a part of Sage Days 60, I was just going through these ticket lists to see if there were easy ones:

Isn't this ticket a wont-fix/invalid kind ticket?

comment:9 Changed 5 years ago by kcrisman

  • Reviewers changed from Jason Grout, Karl-Dieter Crisman to Jason Grout, Karl-Dieter Crisman, Kannappan Sampath
  • Status changed from needs_info to positive_review

comment:10 Changed 5 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.