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

refact(api): migrate from my/session to express-session and connect-mongo #200

Merged
merged 14 commits into from
Jun 29, 2019

Conversation

adrienjoly
Copy link
Member

@adrienjoly adrienjoly commented Jun 29, 2019

Contributes to #122 and #55.

What does this PR do / solve?

Openwhyd was developed on top of a custom / home-made web framework called "my-http". In order to standardise it and reduce tech debt, we intend to migrate to Express.js.

This PR is a first step towards that goal: it replaces the custom "my-session" middleware by a more standard combination: connect-mongo and express-session.

Overview of changes

See list of commits for more details.

@adrienjoly adrienjoly self-assigned this Jun 29, 2019
@adrienjoly adrienjoly added this to 📥 Inbox / ideas in Development via automation Jun 29, 2019
@adrienjoly adrienjoly changed the title refact(api): migrate from my/http to express refact(api): migrate from my/session to express-session and connect-mongo Jun 29, 2019
@adrienjoly adrienjoly merged commit 0a18c5b into master Jun 29, 2019
Development automation moved this from 📥 Inbox / ideas to ✔️ Done / pending QA Jun 29, 2019
@adrienjoly adrienjoly deleted the refact/migrate-to-express branch June 29, 2019 13:12
adrienjoly pushed a commit that referenced this pull request Jun 29, 2019
# [1.8.0](v1.7.2...v1.8.0) (2019-06-29)

### Features

* **api:** migrate from my/session to express-session and connect-mongo ([#200](#200)) ([0a18c5b](0a18c5b)), closes [#122](#122) [#55](#55)
@adrienjoly adrienjoly moved this from ✔️ Done / pending QA to 🌲 In production in Development Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development
  
🌲 In production
Development

Successfully merging this pull request may close these issues.

None yet

1 participant