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

Jump to bottom

#style methods don't work then session cookies are enabled #594

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 2 comments
Closed

#style methods don't work then session cookies are enabled #594

GoogleCodeExporter opened this issue Mar 25, 2015 · 2 comments
Labels
auto-migrated Priority-High Type-Defect

Comments

@GoogleCodeExporter
Copy link 

What steps will reproduce the problem?
1. Enable session cookies for examples/counter
2. Add a #style method to WACounter eg. 'body: {color:red}'
3. To go http://127.0.0.1:8080/examples/counter
4. The style will work because there's a link like this:
<link href="/examples/counter?_s=95tPv585oByGeUxH" type="text/css" 
rel="stylesheet">
5. Now click on '++' or '--' the style will no longer work because there's a 
link like this:
<link href="/examples/counter?_s=95tPv585oByGeUxH" type="text/css" 
rel="stylesheet">

This is a very nasty bug because potentially all senders of 
#urlForDocument:mimeType:fileName: to WARenderContext are broken. I believe the 
right fix would be if WARegistry >> #urlFor:ifAbsent: strips the session key 
only if the request handler is a session.

Original issue reported on code.google.com by philippe...@gmail.com on 13 Aug 2010 at 10:24
@GoogleCodeExporter
Copy link
Author

Original comment by philippe...@gmail.com on 13 Aug 2010 at 12:01

  • Changed state: Started
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author 

Name: Seaside-Core-pmm.655
Author: pmm
Time: 13 August 2010, 2:02:03 pm
UUID: 4e7d671c-eb15-4291-8213-70e7a319c8a2
Ancestors: Seaside-Core-pmm.654

- Issue 594: #style methods don't work then session cookies are enabled
- http://code.google.com/p/seaside/issues/detail?id=594

Name: Seaside-Session-pmm.133
Author: pmm
Time: 13 August 2010, 2:02:55 pm
UUID: 20c413c9-f088-4b94-a361-6496656cd575
Ancestors: Seaside-Session-jf.132

- Issue 594: #style methods don't work then session cookies are enabled
- http://code.google.com/p/seaside/issues/detail?id=594

Name: Seaside-Tests-Core-pmm.198
Author: pmm
Time: 13 August 2010, 2:03:37 pm
UUID: ad1f5f9f-d5d8-419a-a015-28b65af5af5b
Ancestors: Seaside-Tests-Core-pmm.197

- Issue 594: #style methods don't work then session cookies are enabled
- http://code.google.com/p/seaside/issues/detail?id=594

Name: Seaside-Tests-Session-pmm.41
Author: pmm
Time: 13 August 2010, 2:04:19 pm
UUID: 95ec60c6-1e39-4ce0-a985-b6f4d82bdf8b
Ancestors: Seaside-Tests-Session-pmm.40

- Issue 594: #style methods don't work then session cookies are enabled
- http://code.google.com/p/seaside/issues/detail?id=594

Original comment by philippe...@gmail.com on 13 Aug 2010 at 12:04

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated Priority-High Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter