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

[CLOSE THIS][WIP] Completely and permanently replace continuation-local-storage with cls-hooked #10

Closed
wants to merge 14 commits into from
Closed

[CLOSE THIS][WIP] Completely and permanently replace continuation-local-storage with cls-hooked #10

wants to merge 14 commits into from

Commits on Aug 2, 2016

  1. Initial implementation

    @bajtos @josieusa
    bajtos authored and josieusa committed Aug 2, 2016
    Configuration menu
    Copy the full SHA
    c7e8bf7 View commit details
    Browse the repository at this point in the history

  2. Rename to loopback-context/LoopBackContext

    @bajtos @josieusa
    bajtos authored and josieusa committed Aug 2, 2016
    Configuration menu
    Copy the full SHA
    2a0bbb3 View commit details
    Browse the repository at this point in the history

  3. Lazy-load CLS module 

    As soon as CLS module is loaded, the instrumentation/patching of
async-listener is fired. This may cause stack overflows due to promise
instrumentation.

By loading CLS module lazily (only when used for real), we avoid
this kind of problems in applications that are not using current-context
at all.
    @bajtos @josieusa
    bajtos authored and josieusa committed Aug 2, 2016
    Configuration menu
    Copy the full SHA
    df60f13 View commit details
    Browse the repository at this point in the history

  4. fixup! style

    @bajtos @josieusa
    bajtos authored and josieusa committed Aug 2, 2016
    Configuration menu
    Copy the full SHA
    12a7113 View commit details
    Browse the repository at this point in the history

  5. Enable Travis CI

    @bajtos @josieusa
    bajtos authored and josieusa committed Aug 2, 2016
    Configuration menu
    Copy the full SHA
    40672a7 View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2016

  1. API cleanup 

    Rename "per-request-context" middleware to shorter "per-request".

Introduce "LoopBackContext.perRequest" as a shortcut for way too long
  "loopback-context/server/middlewar/per-request"
    @bajtos @josieusa
    bajtos authored and josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    404c27b View commit details
    Browse the repository at this point in the history

  2. Add big fat warning to README 

    Recommend against using this module.
    @bajtos @josieusa
    bajtos authored and josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    4e4ed1d View commit details
    Browse the repository at this point in the history

  3. 1.0.0 

     * First release!
    @bajtos @josieusa
    bajtos authored and josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    1fd90e0 View commit details
    Browse the repository at this point in the history

  4. make warning less ambigious

    @azatoth @josieusa
    azatoth authored and josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    8433e75 View commit details
    Browse the repository at this point in the history

  5. Merge upstream/master into master to sync with upstream repo 

    Fix comment
    @josieusa
    josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    c96e37c View commit details
    Browse the repository at this point in the history

  6. Revert "Lazy-load CLS module"

    @josieusa
    josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    f120f2e View commit details
    Browse the repository at this point in the history

  7. Use cls-hooked instead of continuation-local-storage, 

    as suggested by @raymondfeng
in strongloop/loopback-context PR strongloop#1 comment #235931961
    @josieusa
    josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    38f6110 View commit details
    Browse the repository at this point in the history

  8. Document in README.md and test correct usage of cls-hooked

    @josieusa
    josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    1047af4 View commit details
    Browse the repository at this point in the history

  9. Merge upstream/master into feature/cls-hooked-2 to sync with upstream… 

    … repo
    @josieusa
    josieusa committed Sep 9, 2016
    Configuration menu
    Copy the full SHA
    0c76b55 View commit details
    Browse the repository at this point in the history