Skip to content

Session to client migrations#4412

Merged
philbogle merged 1 commit into
stagingfrom
revert-4299-revert-4283-session_to_client_migrations
Oct 8, 2015
Merged

Session to client migrations#4412
philbogle merged 1 commit into
stagingfrom
revert-4299-revert-4283-session_to_client_migrations

Conversation

@philbogle

Copy link
Copy Markdown
Contributor

Migrate videos_seen, callouts_seen, and scripts from the session cookie to unencrypted cookies accessible from Javascript.

(I rolled back the original version of this PR (#4299) because the needed cookbook changes had not been applied yet. This time around those changes have been deployed all the way to production so I hope to see no further difficulties.)

Reverts #4299 which reverted #4283.

@mehalshah

Copy link
Copy Markdown
Contributor

LGTM

@philbogle

Copy link
Copy Markdown
Contributor Author

I confirmed that the timeout in the CircleCI is a known flaky test and not related to this PR.

For example, this is another such failure:
https://circleci.com/gh/code-dot-org/code-dot-org/78

philbogle added a commit that referenced this pull request Oct 8, 2015
…ssion_to_client_migrations

Session to client migrations
@philbogle philbogle merged commit e25e318 into staging Oct 8, 2015
@philbogle

Copy link
Copy Markdown
Contributor Author

This is still encountering some issues with the CDN as seen via staging, so unfortunately I'm going to have to revert it again. Adhoc also has the CDN set up so I can look there.

@davidsbailey davidsbailey deleted the revert-4299-revert-4283-session_to_client_migrations branch January 7, 2016 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants