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

[Positive review] dsage.setup() add_default_client broken #3314

Closed
yqiang mannequin opened this issue May 27, 2008 · 9 comments
Closed

[Positive review] dsage.setup() add_default_client broken #3314

yqiang mannequin opened this issue May 27, 2008 · 9 comments

Comments

@yqiang
Copy link
Mannequin

yqiang mannequin commented May 27, 2008

twisted changed it's conch interface and I forgot to toString the pubkey object in some instances.

Component: dsage

Keywords: editor_gfurnish

Issue created by migration from https://trac.sagemath.org/ticket/3314

@yqiang
Copy link
Mannequin Author

yqiang mannequin commented May 27, 2008

Attachment: 3314_default_users.patch.gz

@yqiang yqiang mannequin added the s: needs review label May 27, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-3.0.3 milestone May 28, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin assigned yqiang and unassigned sagetrac-mabshoff May 28, 2008
@craigcitro
Copy link
Member

Changed keywords from none to editor_gfurnish

@garyfurnish
Copy link
Mannequin

garyfurnish mannequin commented Jun 16, 2008

comment:5

DSage is completely broken in 3.0.2 and 3.0.3!!!!! Make me some patches so that I can actually run dsage.

@garyfurnish garyfurnish mannequin changed the title dsage.setup() add_default_client broken [Negative review] dsage.setup() add_default_client broken Jun 16, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jun 16, 2008

comment:6

DSage is not completely broken in Sage 3.0.2 with the patch applied and in 3.3.0.rc0. If you make claims like that you should back them up with details.

Cheers,

Michael

@garyfurnish
Copy link
Mannequin

garyfurnish mannequin commented Jun 16, 2008

comment:7

If you do not have an existing known good .sage directory with existing dsage materials from some previous version (unknown), dsage is completely broken.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jun 17, 2008

comment:8

Ok, the bug is the following after deleting the dsage directory in DOT_SAGE:

mabshoff@sage:/scratch/mabshoff/release-cycle/sage-3.0.3.rc0$ ./sage
----------------------------------------------------------------------
| SAGE Version 3.0.3.rc0, Release Date: 2008-06-16                   |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------

sage: dsage.setup()
==================================================
Generating public/private key pair for authentication...
Your key will be stored in /home/mabshoff/.sage/dsage/dsage_key
Just hit enter when prompted for a passphrase
==================================================
<SNIP>
sage: dsage.start_all()
Spawned twistd -d /home/mabshoff/.sage/dsage --pidfile=server.pid --logfile=/home/mabshoff/.sage/dsage/server.log -y dsage_server.tac (pid = 26784)

Spawned python /scratch/mabshoff/release-cycle/sage-3.0.3.rc0/local/bin/dsage_worker.py -s localhost -p 8087 -u mabshoff -w 2 --poll 1.0 -l 0 -f /home/mabshoff/.sage/dsage/worker.log --privkey=/home/mabshoff/.sage/dsage/dsage_key --pubkey=/home/mabshoff/.sage/dsage/dsage_key.pub --priority=20 --ssl --noblock (pid = 26801)

Not connected.
sage: 

The DSage server log says:

2008-06-16 17:08:52-0700 [-] Log opened.
2008-06-16 17:08:52-0700 [-] twistd 8.0.1 (/scratch/mabshoff/release-cycle/sage-3.0.3.rc0/local/bin/python 2.5.2) starting up
2008-06-16 17:08:52-0700 [-] reactor class: <class 'twisted.internet.selectreactor.SelectReactor'>
2008-06-16 17:08:52-0700 [-] twisted.spread.pb.PBServerFactory starting on 8087
2008-06-16 17:08:52-0700 [-] Starting factory <twisted.spread.pb.PBServerFactory instance at 0x2b54f6e88998>
2008-06-16 17:08:52-0700 [-] twisted.web2.channel.http.HTTPFactory starting on 8088
2008-06-16 17:08:52-0700 [-] Starting factory <twisted.web2.channel.http.HTTPFactory instance at 0x2b54f6e88e60>
2008-06-16 17:08:52-0700 [Broker,1,127.0.0.1] <client:'mabshoff'> connected
2008-06-16 17:08:53-0700 [Broker,2,127.0.0.1] <unauthenticated_worker:'Anonymous'> connected
2008-06-16 17:12:03-0700 [Broker,1,127.0.0.1] <client:'mabshoff'> disconnected
2008-06-16 17:12:03-0700 [Broker,2,127.0.0.1] <unauthenticated_worker:'Anonymous'> disconnected

Cheers,

Michael

@yqiang
Copy link
Mannequin Author

yqiang mannequin commented Jun 17, 2008

comment:9

Michael, Gary, did you test this after applying the patch I posted?

@yqiang
Copy link
Mannequin Author

yqiang mannequin commented Jun 17, 2008

comment:10

Ok. I tried this with 3.0.3rc0 after applying both #3312 and #3314 and can't repro this.

@garyfurnish garyfurnish mannequin changed the title [Negative review] dsage.setup() add_default_client broken [Positive review] dsage.setup() add_default_client broken Jun 18, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jun 18, 2008

comment:13

Merged in Sage 3.0.3.final

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.0.4, sage-3.0.3 Jun 18, 2008
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

2 participants