Skip to content

Added context to App facade constructor - #569

Closed
Minifets wants to merge 3 commits into
ratchetphp:v0.4.4from
Minifets:master
Closed

Minifets wants to merge 3 commits into
ratchetphp:v0.4.4from
Minifets:master

Conversation

@Minifets

Copy link
Copy Markdown
Contributor

This will help to create the tls server.

For example:
$app = new App('localhost', 8080, 'tls://0.0.0.0', $loop, ['tls' => [ 'allow_self_signed' => true, 'local_cert' => '/etc/nginx/ssl/localhost.crt', 'local_pk' => '/etc/nginx/ssl/localhost.key' ]]);

Comment thread composer.json Outdated
@@ -1,5 +1,5 @@
{
"name": "cboden/ratchet"
"name": "minifets/ratchet"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think you forget to remove this diff :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I added the changes after the submiting pull request.

This reverts commit 5ebe073.
@cboden cboden added this to the 0.4.1 milestone Oct 24, 2017
@cboden cboden modified the milestones: 0.4.1, 0.4.2 Dec 11, 2017
@mbonneau mbonneau modified the milestones: 0.4.2, 0.4.3 Jan 28, 2020
@cboden cboden modified the milestones: 0.4.3, 0.4.4 Jun 4, 2020
@cboden
cboden changed the base branch from master to v0.4.4 December 12, 2021 16:45
@cboden

cboden commented Dec 12, 2021

Copy link
Copy Markdown
Member

merged

@cboden cboden closed this Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants