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.

Sign up for GitHub

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

Jump to bottom

change cookies structure to support multiple notebook logins at different ports #6353

Closed
boothby opened this issue Jun 18, 2009 · 6 comments
Closed

change cookies structure to support multiple notebook logins at different ports #6353

boothby opened this issue Jun 18, 2009 · 6 comments
Assignees
@boothby
Labels
c: user interface p: minor / 4 t: bug
Milestone
sage-4.3.2

Comments

@boothby
Copy link

boothby commented Jun 18, 2009

At present, a user cannot log into two different notebooks at the same address with different ports. For example, if I run two notebooks at

http://sage.math.washington.edu:8001

and

http://sage.math.washington.edu:8002

then I can only use one at a time unless I use two separate browsers. A simple change to the cookies should fix this.

CC: @boothby

Component: notebook

Author: Tim Dumol

Reviewer: Mitesh Patel

Merged: sagenb-0.7

Issue created by migration from https://trac.sagemath.org/ticket/6353
@boothby boothby added this to the sage-4.3.2 milestone Jun 18, 2009
@boothby boothby self-assigned this Jun 18, 2009
@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 19, 2010

Attachment: trac_6353-cookies-diff-ports.patch.gz

Appends port number to the cookie names.

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 19, 2010

comment:1

This should do the trick.

@TimDumol TimDumol mannequin added the s: needs review label Jan 19, 2010
@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 20, 2010

comment:2

Nice excuse to try out Firecookie.

@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 20, 2010

Reviewer: Mitesh Patel

@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 20, 2010

Author: Tim Dumol

@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 25, 2010

Merged: sagenb-0.7

@qed777 qed777 mannequin removed the s: positive review label Jan 25, 2010
@qed777 qed777 mannequin closed this as completed Jan 25, 2010
@qed777 qed777 mannequin mentioned this issue Feb 10, 2010
Closed
@williamstein williamstein mentioned this issue May 4, 2010
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@boothby boothby

Labels
c: user interface p: minor / 4 t: bug
Projects
None yet
Milestone
sage-4.3.2
Development

No branches or pull requests
1 participant
@boothby