Skip to content

Igore excessive fopen() mode flags for WritableResourceStream - #119

Merged
WyriHaximus merged 2 commits into
reactphp:masterfrom
clue-labs:flags
Nov 10, 2017
Merged

Igore excessive fopen() mode flags for WritableResourceStream#119
WyriHaximus merged 2 commits into
reactphp:masterfrom
clue-labs:flags

Conversation

@clue

@clueclue commented Nov 9, 2017

Copy link
Copy Markdown
Member

This fixes a minor issue where valid stream resources opened with valid flags such as re where not properly rejected.

Builds on top of #83, #84, #85

@clueclue added the bug label Nov 9, 2017
@clueclue added this to the v0.7.5 milestone Nov 9, 2017
@clue

clue commented Nov 9, 2017

Copy link
Copy Markdown
MemberAuthor

For the reference: I consider this a low priority bug that I stumbled over while evaluating the e fopen flag for reactphp/child-process#51.

Comment threadsrc/WritableResourceStream.php
@clue

clue commented Nov 9, 2017

Copy link
Copy Markdown
MemberAuthor

Unrelated Mac OS X test failures are being addressed in #120:shipit:

@WyriHaximusWyriHaximus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@WyriHaximus
WyriHaximus merged commit a179ab5 into reactphp:masterNov 10, 2017
@clue
clue deleted the flags branch November 10, 2017 16:58
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@clue@jsor@WyriHaximus@kelunik