Skip to content

add client timeout #26

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from
Closed

add client timeout #26

wants to merge 1 commit into from

Conversation

61-6c-69
Copy link

No description provided.

@@ -29,4 +29,9 @@
* The URL to the NativePHP API.
*/
'api_url' => env('NATIVEPHP_API_URL', 'http://localhost:4000/api/'),

/**
*
Copy link
Contributor

Choose a reason for hiding this comment

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

Add some description!

@mpociot
Copy link
Member

mpociot commented Jul 21, 2023

To be honest, I don't think this should be configurable.
We can move it to a constant to add some context to it, but it does not make sense to lower this as this is only used between the internal API between PHP and Electron.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants