2024-08-29 Meeting notes

@Craig McNally 

@Julian Ladisch 

Sep 19, 2024 time collision

@Axel Dörrer 

@Ryan Berger 

@Chris Rutledge 

@Jakub Skoczen 

@John Coburn 

@Skott Klebe 

@Kevin Day

@Jens Heinrich

Out next week Aug 29, 2024 

5 min

Proxy for Dev-Setup

 John + Team

• John has been discussing this with Zak/Jason - doing this in stripes-cli.  It sounds like they're coming around to this idea.

• A spike is still needed

  • @John Coburn will create the spike and ask for input/help from the rest of the Security team as needed.

Today:

• @John Coburn is unable to attend today.  Let's check in next week.

5 min

Cookie SameSite topic raised in Slack

Team + Zak

• Only remaining gotcha is how this will work with STSH (strict transport security headers)

• @John Coburn created a ticket to include CORS in the proxy

  • https://folio-org.atlassian.net/browse/STCLI-247

  • @Kevin Day will test something by Zak here

• TAMU is using an older version of Okapi which is making things more complicated, Kevin is making due.

Today:

• Update from @John Coburn via slack:

  • "I believe Zak set up a branch with CORs updates to stripes-cli. Not sure if it's been merged yet." 

• There is a PR linked in the Jira above, and it is not merged yet.  https://github.com/folio-org/stripes-cli/pull/351

• @Kevin Day would like to suggest not making this specific to localhost:3000

  • See https://github.com/folio-org/stripes-cli/compare/master...kaladay:stripes-cli-2.x-backports:proxy-strip-acao-custom

1 min

Update on SECURITY-159 / FOLIO-4012

Craig

We're on track here.  Ian H. applied mitigation.  Upgrade will happen when John Malconian returns from vacation.

• @Jakub Skoczen to circle back and check in with John M.

• Craig posted a reminder in #devops

• Check in on this next week Aug 1, 2024

• @Jakub Skoczen informed that the devops team will pick this up soon.

• @Craig McNally reached out to @John Malconian and @Jakub Skoczen for an update today.  

  • Response from John: 'I’ll be starting this soon.  Typically will need to be done during “off hours” so I just need to find a good time.'

Today

• @Craig McNally will nudge John Malconian again.  DONE.

Response from John on Aug 22, 2024 :

the ID side of the DevOps team is in a state of flux. We are in the process of hiring a DevOps person who can dedicate a certain percentage of their time to FOLIO Devops.  At the moment,  we are stretched thin and do what we can which is mostly triage at the moment.https://folio-org.atlassian.net/browse/FOLIO-4012  Still need to schedule this, but aiming to complete before Sept 1.

NOTE:  Sept 1 is this Sunday, and next week is a holiday in the US, so I guess we'll just check in with John again next week?

0 min

Jira Group and Security Level review

Team

From Craig in slack:

I've been in communication with David Crossley, Wayne Schneider, John Malconian and Peter Murray about the issue above.  They apparently didn't have access to these embargoed issues (SysOps and Core Team).  Peter shared this screenshot with me, which doesn't look right.  I'd like to review this at one of our meetings and come up with a list of changes/improvements for Peter to make.  A few ideas off the top of my head:

1. Add descriptions to each of the security groups, like we have for "FOLIO Security Group"

2. Maybe add a new security group and level for FOLIO devops

3. Review membership of each of these groups and remove users no longer on the project

4. Review the Security Level -> Group mappings.  Some of these don't look quite right to me.

https://files.slack.com/files-pri/T1EPYQDAQ-F075P5YMBJQ/image.png

• If it makes this easier, we could invite Peter to a meeting so we can see the groups/levels interactively and makes adjustments as we go

TODO:

• Create a new group:  devops, and assign Wayne, John, Ian, David, Peter.  Also create a new security level for "folio devops"

• Remove Johannes Drexl from the SysOps group, he's not longer involved in Folio.

• Action: Craig to look into how security level configuration works to gain a better understanding of why it behaves the way it does.  

  • Why do we have All Folio Developers in the "Sys Ops and Core Team" security level?

Today:

• No update today- Check in on this Sep 5, 2024 

< 5 min

https://folio-org.atlassian.net/browse/FOLIO-3896

Team

• We need to determine if DevOps can work on  or if they don't have bandwidth, and the debian packages aren't used anymore, archive them.  It sounds like some SysOps do use this, but we know that DevOps has very little bandwidth these days.

Today:

• @Jakub Skoczen any update on this?

25 min

Anything Urgent?
Review Mike's Kanban board?
Review Security board?

Under Review Filter: https://folio-org.atlassian.net/issues/?jql=filter%3D12879+

Team

• Action:  Create a Jira for folio devops to set Strict Transport Security header in hosted reference environments.  Then link the story tohttps://folio-org.atlassian.net/browse/SECURITY-170

  • @Craig McNally will create the story and link it up. DONE

  • See https://folio-org.atlassian.net/browse/FOLIO-4049

• Unembargoed several issues which were rejected/completed (SECURITY-171-178

  • https://folio-org.atlassian.net/browse/SECURITY-177 remains open.  See also https://folio-org.atlassian.net/browse/SECURITY-182

    • Action:  Craig will provide updates via slack as Taras digs into this.

• Discussed https://folio-org.atlassian.net/browse/SECURITY-171.  Will keep on the board for tracking purposes.

  • Action: @Julian Ladisch to create a brief wiki page listing 3rd party dependencies/frameworks/etc. which are out of support.  Maybe we should link to this from the flower release notes? DONE (or at least started!)

Today:

• https://folio-org.atlassian.net/browse/SECURITY-180 

  • @Julian Ladisch is creating stories and PRs to address this in the master branch (Ramsons)

  • If we find vulnerabilities, we may want/need to backport this to Quesnelia and possibly Poppy as well.

  • See Dependencies out of support#Spring-5 for additional details, including a GH search which shows the affected repos

• SECURITY-56: CVE-2023-20861. spring-web SpEL DoS - Analysis of vulnerabilityCompletedPreview 

  • The two linked stories are closed, but we need to double check that the version of grails they upgraded to (and other deps) is sufficient to address the vulnerability.

• SECURITY-65: CVE-2023-45648. tomcat-embed-core HTTP request smuggling - Analysis of vulnerabilityCompletedPreview

  • Should be addressed in Quesnelia, but has not yet made it into a Poppy CSP.

  • Since this spans multiple repos/teams it's our responsibility to seek approval for this.

  • Further discussion is required.  Do we really think this is worth pushing for a Poppy CSP?  Who is even affected?

Time permitting

Advice for handling of sensitive banking information

Team

From slack conversation, I think I've gathered the following:

• In this case (bank account and transit numbers), the information is highly sensitive.  

  • Highly sensitive information should:

    • Be stored in it's own table

    • Accessed via a dedicated API

    • Protected by a dedicated permission

    • Encrypted in the database, not only on disk.  

      • This could mean either:

        • Explicitly encrypting/decrypting in the application layer and storing the encrypted data in postgres

        • Utilizing postgres extensions/modules to encypt certain columns, e.g. via pgcrypto (see https://www.postgresql.org/docs/current/encryption-options.html)

Let's review and discuss before providing this feedback to Raman.

@Axel Dörrer also suggested that defining classes of sensitivity could help teams determine which techniques are applicable in various situations.  I agree having some general guidelines on this would be helpful.

• regular data

  • low sensitive - permission based on same API

  • high sensitive - permission based on dedicated API

It would probably help to provide concrete examples of data in each class.  This can be a longer term effort, we don't need to sort out all the details today.

• Next Steps:

  • Clearly define/formalize the various classes

    • Come up with concrete examples of each class

    • Build out guidance

      • Come up with concrete examples of how to protect each class of data.

    • Consider storing some classes of data outside of postgres altogether - e.g. in secret storage.

      • What would be the guidance we provide to teams for this so we don't end up with each team doing things differently?

      • SecretStore interface and existing implementations are currently only read-only.  They would need to be extended to allow for creation/mgmt of this information.

    • Craig to start a conversation in slack about this.

      • Seeking a volunteer to generate a draft document for us to review at a later meeting.

Today:

@Axel Dörrer to do a first draft as a base for further discussions

Status on pentesting works within Network traffic control group

@Axel Dörrer 

Due to some absences on different reasons the group stalled. Axel will try to reactivate the group.