Skip to content
This repository was archived by the owner on Jul 31, 2022. It is now read-only.

Fixed #69 Throttled handler not being removed correctly #70

Merged
merged 1 commit into from
Jul 4, 2020

Conversation

Katsute
Copy link
Member

@Katsute Katsute commented Jul 4, 2020

Prerequisites

If all checks are not passed then the pull request will be closed

  • My code follows the style listed in CONTRIBUTING
  • I have checked that no other similar pull requests already exists
  • Build compiles
  • Build runs without exceptions
  • I have added/modified documentation (if applicable)

Changes Made

list changes made and/or relevant issues

@Katsute Katsute added bug Something isn't working as expected documentation Improvements or additions to the documentation labels Jul 4, 2020
@Katsute Katsute merged commit 38b6abf into master Jul 4, 2020
@Katsute Katsute deleted the fix-temporay-handler@c1cf2b5 branch July 4, 2020 00:52
Katsute added a commit that referenced this pull request Jul 4, 2020
@Katsute Katsute changed the title Fix #69 optional, fix remove context issue Fixed #69 Throttled handler not being removed correctly Jul 4, 2020
Katsute added a commit that referenced this pull request Jul 4, 2020
* NPE (#67)

Co-authored-by: Katsute <[email protected]>

* typo (#68)

Co-authored-by: Katsute <[email protected]>

* Update versioning

* Fix #69 optional, fix remove context issue (#70)

Co-authored-by: Katsute <[email protected]>

* Update SimpleHttpExchangeImpl.java

Co-authored-by: Katsute <[email protected]>
Katsute added a commit that referenced this pull request Jul 4, 2020
* server test case

* fix ports

* Fix ports again

* Server tests

* fixes

* Add context tests

* Added exchange methods

* session & cookie test

* handler tests + #stop

* Create PredicateHandlerTests.java

* cleanup & planning

* Update SSEHandlerTests.java

* NPE (#67)

Co-authored-by: Katsute <[email protected]>

* typo (#68)

Co-authored-by: Katsute <[email protected]>

* Update versioning

* Fix #69 optional, fix remove context issue (#70)

Co-authored-by: Katsute <[email protected]>

* Adapt temporary handler tests

* Update ThrottledHandlerTests.java

* Clean up

* Add tests for removal hotfix

* setup

* SSE tests

* Cleanup 🧹

* Delete file.txt

* Added multipart/form-data test

* Merge latest with master (#73)

* NPE (#67)

Co-authored-by: Katsute <[email protected]>

* typo (#68)

Co-authored-by: Katsute <[email protected]>

* Update versioning

* Fix #69 optional, fix remove context issue (#70)

Co-authored-by: Katsute <[email protected]>

* Update SimpleHttpExchangeImpl.java

Co-authored-by: Katsute <[email protected]>

* Add files / adapter tests +UTF8 warning fix

* Add walking tests

* Update FileHandlerTests.java

* File loading option tests 🏁

Co-authored-by: Katsute <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working as expected documentation Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Temporary handler doesn't remove context
1 participant