2024-11-21 Meeting notes

Craig McNally 

Axel Dörrer 

Chris Rutledge 

Jakub Skoczen 

Skott Klebe 

• John has been discussing this with Zak/Jason - doing this in stripes-cli.  It sounds like they're coming around to this idea.
• (https://github.com/folio-org/stripes-cli/pull/358 PR has been merged.

Today:

• Is documentation needed?

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
• Not exactly this, but related... 
  • Issues submitted to the SECURITY JIRA project should automatically be embargoed (Security Level = Folio Security Group)
  • The submitter of issues to the SECURITY JIRA project should be able to view issues they submit, regardless of their Security Level
  • Email notification sent to the Folio Security Group when an issue is created in the SECURITY Jira project.
• Action:  Craig McNally to setup a meeting with Peter and representatives from the Security Team to work through these things after WOLFcon?  
  • We need 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:

• (Craig) no progress due to lack of time to spend on this.  Does someone else have time to work through this with Peter?

Anything Urgent?
Review Mike's Kanban board?
Review Security board?
Review labels=security?

Under Review Filter:  Getting issues...

• Snyk review - criticals!
• mod-reporting has 5 critical issues
  • Uses Go language
  • TC has officially decided that FOLIO will support Go language for module development.  The OST pages have not yet been updated.
  • Would need for Go language to be approved and mod-reporting needs to be approved by TC to be included in Sunflower release
  • Other modules that are used in production are higher priority to address
• mod-graphql has 1 critical issue
  • Mike Taylor is the owner. Need to reach out to see if he can fix the issue. If he is not available, need to go through TC. MODGQL-160 - Getting issue details... STATUS
• SECURITY-218 - Getting issue details... STATUS was discussed.  Eureka team should provide details.  Craig McNally will coordinate.

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.

Axel Dörrer 

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