Netgate Discussion Forum
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Search
    • Register
    • Login

    How to get Feedback on PRs

    Scheduled Pinned Locked Moved Development
    11 Posts 4 Posters 589 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • M
      MatthewA1
      last edited by

      I've been waiting 6 months for feedback on my PR #4658 for Redmine ticket #8794. Is there some process I'm not seeing for requesting feedback? At this point, the PR has conflicts that I'll need to correct, but I don't want to spend the time to do so if it's just never going to be reviewed. Is there someone to contact about this? Posting comments on both the PR and Redmine ticket hasn't resulted in any responses since the PR was initially opened.
      If I just need to wait, that's fine, but at least some timeline or basic response would be appreciated.

      1 Reply Last reply Reply Quote 1
      • M
        marcosm Netgate
        last edited by marcosm

        I'm happy to see people contributing to the project! The only thing left to do in that situation is wait; unfortunately it can take a while to get to these simply because a lot of the time there are other priorities that we need to focus our effort on.

        M 1 Reply Last reply Reply Quote 1
        • M
          MatthewA1 @marcosm
          last edited by

          Thanks @marcosm , you actually reviewed that PR a couple days ago. I had just wanted to make sure it hadn't gotten lost or it was waiting on something else from me.
          One follow-up question: As discussed in the comments of the PR, there's more work to be done in a future PR (such as supporting multiple keys). As the Redmine ticket is in the Feedback status and shows the work is 100% done plus it is simply titled "Support NTP Authentication," should I start that work under the same Redmine Ticket, or since it is improving what is now a supported feature, should I open a new Redmine Ticket first?

          M 1 Reply Last reply Reply Quote 0
          • M
            marcosm Netgate @MatthewA1
            last edited by

            @MatthewA1 It'd be best to put it in a new feature request that references the old one.

            M 1 Reply Last reply Reply Quote 2
            • JonathanLeeJ
              JonathanLee
              last edited by

              What about Squid it is updated I see but certification issues with OpenSSL exist still. I am taking C next semester, I can't wait to start helping out more. They have set me up for full computer science major now, it is no longer pre computer science WHooHoo

              Make sure to upvote

              M 1 Reply Last reply Reply Quote 0
              • M
                marcosm Netgate @JonathanLee
                last edited by marcosm

                @JonathanLee There's no maintainer for the squid packages on pfSense - fixes/features are unlikely to be released for it. Until it's removed, any updates for it will likely be general maintenance patches.

                JonathanLeeJ 1 Reply Last reply Reply Quote 1
                • JonathanLeeJ
                  JonathanLee @marcosm
                  last edited by

                  @marcosm Someone updated Squid from 5.8 to 6.X recently.

                  Make sure to upvote

                  M 1 Reply Last reply Reply Quote 0
                  • M
                    MatthewA1 @marcosm
                    last edited by

                    @marcosm Opened as requested #15579
                    If the Assignee field is used for anyone (not just Netgate employees), feel free to assign to me.

                    1 Reply Last reply Reply Quote 1
                    • M
                      michmoor LAYER 8 Rebel Alliance @JonathanLee
                      last edited by

                      @JonathanLee He is stating there is no pfsense maintainer.
                      There’s the FreeBSD package maintainer and a Pfsense package maintainer.
                      If either one of these maintainers no longer have interest in a project it withers on the vine.
                      So yes Squid has been updated but unless there’s a hot fix to a security issue the package is done on pfsense

                      Firewall: NetGate,Palo Alto-VM,Juniper SRX
                      Routing: Juniper, Arista, Cisco
                      Switching: Juniper, Arista, Cisco
                      Wireless: Unifi, Aruba IAP
                      JNCIP,CCNP Enterprise

                      JonathanLeeJ 2 Replies Last reply Reply Quote 1
                      • JonathanLeeJ
                        JonathanLee @michmoor
                        last edited by JonathanLee

                        @michmoor the 6.6 update is the security fix. It’s the latest version. Squid did fix a lot of things in the latest version of Squid and it is downloading in packages. I have this feeling big tech wants it dead because it works so well. It’s sad no reason other than the fact that it flys in the face of big tech abuse. It’s like a giant spot light.

                        Squid has so much to offer, it just needs some fresh code thrown at it again. I really want a container analysis system, that way it wouldn’t matter if a system has an invasive container the proxy it would see the fingerprints and alert cyber security services about the situation. That’s what cyber security needs. A workable mitigation for container and virtual machine issues, FreeBSD jails, invasive actors. A spotlight for invasive actors that are performing abusive data marshaling. There is got to be a solution, and proxy software that caches and accelerates is the answer to it, again it needs code and time.

                        Make sure to upvote

                        1 Reply Last reply Reply Quote 0
                        • JonathanLeeJ
                          JonathanLee @michmoor
                          last edited by

                          @michmoor you know big tech says, They say : "Squid is dangerous ..." You want to maintain it with me? We just need to fix the gui is all

                          Make sure to upvote

                          1 Reply Last reply Reply Quote 0
                          • First post
                            Last post
                          Copyright 2025 Rubicon Communications LLC (Netgate). All rights reserved.