Skip to content
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

Accessing sensitive information policy #25

Closed
wants to merge 3 commits into from

Conversation

TamaraDale
Copy link
Contributor

Also including related tables for review

@TamaraDale TamaraDale self-assigned this Jul 20, 2022
@TamaraDale TamaraDale added the policy change A change to a policy is being proposed label Jul 20, 2022
@mattcaswell
Copy link
Member

We only seem to have the tables as an ods file in this PR - not the actual policy itself.

Is an ods file the best format for publishing the tables? I wonder if it would be better done in markdown? Also should we hold the table in general-policies, or should that be held elsewhere?

@levitte
Copy link
Member

levitte commented Jul 20, 2022

There are two sheets in there, SIT and SIAT... If I understand correctly, SIT is some start of a policy, while SIAT is more of an information table that shows the details. Is that about right, @TamaraDale?

Regarding the format, our Policy on Proposing General Policy Changes is quite clear, at least regarding policies:

"Each policy is placed in an individual file in Markdown format in the policies subdirectory."

Informational tables haven't been thought of, at least to have here, so we haven't said much about their format... but yeah, they should be in a different directory.

@mattcaswell
Copy link
Member

If I understand correctly, SIT is some start of a policy, while SIAT

The SIT is what information we have, and the SIAT is who is allowed to access it.

Informational tables haven't been thought of, at least to have here, so we haven't said much about their format... but yeah, they should be in a different directory.

I was comparing this to the platform policy. Previous discussions on this were that the policy itself would be in general-policies (when it is eventually moved there), but the table of platforms would be maintained outside of the general-policies repo because its not considered part of the policy itself and changes much more frequently. I was applying that same logic to this and wondering whether there is a distinction between the policy which we assume to be relatively long term stable - and the tables which might be expected to change more frequently.

@TamaraDale
Copy link
Contributor Author

TamaraDale commented Jul 21, 2022 via email

Copy link
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer the tables to be converted to markdown - but I could accept them as they are.

In the SIAT the entry for System Administrator seems incomplete: "Passwords for"

@@ -0,0 +1,13 @@
| **Sensitive Information Access Table** | |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought these were going into a new folder one level up. Attachments/schedule/data/whatever.

@TamaraDale
Copy link
Contributor Author

TamaraDale commented Sep 30, 2022 via email

@paulidale
Copy link
Contributor

Sounds fine, although it might be better to use something more generic in case other policies need to reference things that aren't tables. Still, the name can be changed later easily enough.

@TamaraDale
Copy link
Contributor Author

TamaraDale commented Sep 30, 2022 via email

@paulidale
Copy link
Contributor

Yep, or even just supplemental since this is the general-policy repository.

@@ -0,0 +1,13 @@
| **Sensitive Information Access Table** | |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make this a header:

Sensitive Information Access Table
==================================

@paulidale
Copy link
Contributor

i.e.

Sensitive Information Access Table
==================================

| Role/Individual 	| Accessible Sensitive Information 	|
| --- | --- |

Also including related tables for review
@paulidale paulidale force-pushed the AccessSensitiveInfo branch 2 times, most recently from 5a1674c to 71db0cd Compare November 10, 2022 22:21
@paulidale paulidale added the ready to vote The policy change proposal is ready to be voted on by OMC label Nov 10, 2022
@paulidale
Copy link
Contributor

Vote: Accept the accessing sensitive information policy as of 2894caf

@paulidale
Copy link
Contributor

Pauli: [+1]

@mattcaswell
Copy link
Member

Vote: [+1]

@iamamoose
Copy link
Member

Vote: 0

@t-j-h
Copy link
Member

t-j-h commented Nov 15, 2022

Vote [+1]

There is stuff I would handle differently - but we need to start somewhere ...

@kroeckx
Copy link
Member

kroeckx commented Nov 22, 2022

voting +1

@levitte
Copy link
Member

levitte commented Nov 30, 2022

vote: [0]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
policy change A change to a policy is being proposed ready to vote The policy change proposal is ready to be voted on by OMC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants