Skip to content

[pull] master from opnsense:master#1270

Open
pull[bot] wants to merge 4000 commits intomarcushoff:masterfrom
opnsense:master
Open

[pull] master from opnsense:master#1270
pull[bot] wants to merge 4000 commits intomarcushoff:masterfrom
opnsense:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Nov 27, 2023

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull Bot added ⤵️ pull merge-conflict Resolve conflicts manually labels Nov 27, 2023
@AdSchellevis AdSchellevis force-pushed the master branch 3 times, most recently from 272532c to 43a1743 Compare January 4, 2024 20:12
@fichtner fichtner force-pushed the master branch 2 times, most recently from 89d5c6b to f5ae0c8 Compare January 30, 2024 20:26
@AdSchellevis AdSchellevis force-pushed the master branch 2 times, most recently from 709906d to bc2ca23 Compare March 17, 2024 12:36
@AdSchellevis AdSchellevis force-pushed the master branch 5 times, most recently from 3d71277 to 20a8006 Compare May 4, 2024 18:34
@AdSchellevis AdSchellevis force-pushed the master branch 6 times, most recently from 78845fc to 8ba454a Compare May 22, 2024 15:53
@AdSchellevis AdSchellevis force-pushed the master branch 2 times, most recently from 810ecdc to 897d3cc Compare July 26, 2024 08:31
fichtner and others added 30 commits May 5, 2026 14:17
…lter_var(..,FILTER_VALIDATE_EMAIL) might align more with RFC's, but since RFC 5322 accepts almost everything, might not be the best option in reality.
… have changed the configuration (group modification), closes #10255
…10258)

Align implementation with get_real_interface() except that we shall
not give a fallback of $device = $interface since we want better
integrity in MVC code (and don't deal with devices in configuration
data there).
… have changed the configuration (group modification), closes #10255  [part 2]

Missed the fact that $config and $userindex should be globals here due to legacy reasons and missed the index_users() call which is also used in diag_authentication.php
…l ip's, which sometimes collisions with our standard handling when it comes to "nonsync" behavior.

In f469cf8 we fixed virtual ips with a vhid set, but missed the point that the backup may also set nosync on an item not in the list at all, in which case these items that are not in the list will still be duplicated.

This commit removes the vhid constraint and ignore all "nosync" records, which should close the gap here.
This is a fringe use case, but it's better to do this here
than to handroll upcoming changes in the ipsec connections page.
Co-authored-by: Stephan de Wit <stephan.de.wit@deciso.com>
Introduce the base apply button to the page and improve
UX and UI:

1. Local/remote now show ID
2. Tabbed dialog with cancel button and better save/cancel
   placement
3. Better grid spacing in tabbed dialog
4. Apply now closes the tabbed dialog, but save does not
5. Avoid grid flapping in tabbed dialog

Co-authored-by: Stephan de Wit <stephan.de.wit@deciso.com>
Discussed-with: @Monviech
It's no longer required since the static PHP pages are gone and
config_read_array() doesn't need it either.
A major change for later.  Luckily only Kea ever (almost) used this.
This reverts commit 7c8bda9.

This didn't add any value.  But keep the explode->getValues transformation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⤵️ pull merge-conflict Resolve conflicts manually

Projects

None yet

Development

Successfully merging this pull request may close these issues.