forked from pfsense/pfsense
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refresh Fork #1
Merged
Merged
Refresh Fork #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* DNS Resolver python integration
* Add changes as requested by @jim-p
* Include any additional functions as defined by python script include file * Add missing escapeshellarg()'s * Make grep mount validation cmd more specific to include "(nullfs"
It seems DNSimple started using HTTP/2, which broke the regex the dnsimple updater was using to check for success. I changed it to use the CURLINFO_HTTP_CODE instead. I noticed several other providers are using the regex status match instead of HTTP_CODE, but I didn't touch those. I haven't written any PHP in 20 years, and I don't use those providers to test them. This change works for me for DNSimple now. The HTTP/2 response the API was sending back (missing the "OK" the regex was looking for): ``` HTTP/2 200 \r\n server: nginx ... ``` Fixes Ticket #9580
* Sanitize user input before using as path/filenames * Use a more accurate method of determining image type on read * More sanity checks before reading images.
This reverts commit 987377b.
Not an ideal solution but it does ensure that FRR routes function after an IPsec event.
…tead. Fixes #9668
…So it was saving the asterisks and overwriting the current password.
Remove all referenes to legact/Gold ABC system
…s #9522 While here, set a high number of processes to display and also remove the use of 'cut' which limited column length. With current display method it's unnecessary.
* Move code to function to avoid unnecessary duplication of code * Clean up the logic to avoid further redundancies * Set keyid type to be quoted and to have its type prefixed
…to provide it in JSON format
This reverts commit 8129d78.
While here, fix log size description to fit new behavior.
* Add rotation count GUI option and per-log option * Add settings for packages to override more fields not supported in the GUI
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.