Skip to content

SecureStream is hard coded into StreamEncryption #30

Closed
Closed
SecureStream is hard coded into StreamEncryption#30
Labels
help wantednew feature

Description

@DangerLifter
DangerLifter
opened on Mar 25, 2015

SecureStream is hard coded into StreamEncryption and there is no easy ability to replace it with extended class.
Reason for replacement: I need to catch and correctly proceed exception " stream_get_contents(): SSL: Connection reset by peer " in method SecureStream::handleData();
Or would be nice updte code with correct catch;

        try {
            $this->handleData($stream);
        } catch (\Exception $e) {
            $this->emit('error', array($e, $this));
        }

\React\Stream\Stream

Activity

  1. cboden commented on May 14, 2015

    @cboden
    cboden
    on May 14, 2015
    Contributor

    Could you provide some code that triggers this error you're having? We're currently looking into issues with stream_get_contents but are having difficulty reproducing errors.

  2. clue commented on Mar 14, 2016

    @clue
    clue
    on Mar 14, 2016
    Contributor

    This issue will likely also be addressed via reactphp/stream#42 and reactphp/stream#40.

  3. clue
    added
    new feature
    help wanted
    on Nov 15, 2016

  4. clue commented on Jan 9, 2017

    @clue
    clue
    on Jan 9, 2017
    Contributor

    Closed via #72

  5. clue
    closed this as completedon Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    help wantednew feature

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions