megahertz / electron-log

Simple logging module Electron/Node.js/NW.js application. No dependencies. No complicated configuration.
MIT License
1.32k stars 128 forks source link

Add remote.sendRequestFn to override log sending #352

Closed megahertz closed 1 year ago

megahertz commented 1 year ago

Discussed in https://github.com/megahertz/electron-log/discussions/351

Originally posted by **psiservices-ccounterman** March 1, 2023 I'm having problems with people running my electron app behind a proxy. I've fixed some issues by using https://www.electronjs.org/docs/latest/api/net which "is similar to the [HTTP](https://nodejs.org/api/http.html) and [HTTPS](https://nodejs.org/api/https.html) modules of Node.js but uses Chromium's native networking library instead of the Node.js implementation, offering better support for web proxies." I'm fixing more, but realized that it is affecting the logging as well. src/transports/remote.js uses http and https. Before I do anything difficult, is there a currently-supported way to swap in the electron net API or substitute a different remote transport? Is there a reason electron-log shouldn't use electron.net as the norm?
megahertz commented 1 year ago

Will be published in 5.0.0-beta.21