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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding security policy #6666

Merged
merged 3 commits into from
May 11, 2023
Merged

Adding security policy #6666

merged 3 commits into from
May 11, 2023

Conversation

sfc-gh-hpathak
Copy link
Collaborator

@sfc-gh-hpathak sfc-gh-hpathak commented May 10, 2023

馃摎 Context

Please describe the project or issue background here

  • What kind of change does this PR introduce?

    • Bugfix
    • Feature
    • Refactoring
    • Other, please describe:

馃 Description of Changes

  • Adding a security policy

  • This is a breaking API change

  • This is a visible (user-facing) change

Revised:

Insert screenshot of your updated UI/code here

Current:

Insert screenshot of existing UI/code here

馃И Testing Done

  • Screenshots included
  • Added/Updated unit tests
  • Added/Updated e2e tests

馃寪 References

Does this depend on other work, documents, or tickets?

  • Issue: Closes #XXXX

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@sfc-gh-hpathak
Copy link
Collaborator Author

@vdonato @kmcgrady I added the supported version , which is the only thing that is different than connectors, lmk if it's accurate or if we even need it.

Copy link
Collaborator

@vdonato vdonato left a comment

Choose a reason for hiding this comment

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

LGTM! Note that I had to include this file in the license header exceptions + the autoformatter edited the file slightly

@vdonato vdonato merged commit 0ed20b1 into develop May 11, 2023
39 checks passed
@vdonato vdonato deleted the security-policy branch May 11, 2023 22:59
tconkling added a commit to tconkling/streamlit that referenced this pull request May 12, 2023
tconkling added a commit to tconkling/streamlit that referenced this pull request May 16, 2023
* develop:
  Cache script bytecode (streamlit#6673)
  Preserve multiple whitespaces in `st.json` rendering of k:v pairs (streamlit#6663)
  Implement required parameter for `st.data_editor` (streamlit#6695)
  Fix typo in Arrow.proto (streamlit#6693)
  Python requirements: pin to < NEXT_MAJOR_VERSION (streamlit#6691)
  Update data_editor docstring re: supported types (streamlit#6650)
  Adding security policy (streamlit#6666)
  Fix import name (streamlit#6669)
  Refactor Duplicated Export Names (streamlit#6660)
eric-skydio pushed a commit to eric-skydio/streamlit that referenced this pull request Dec 20, 2023
* Create SECURITY.md

* Don't require license header on SECURITY.md

* Autoformat SECURITY.md

---------

Co-authored-by: Vincent Donato <vincent@streamlit.io>
zyxue pushed a commit to zyxue/streamlit that referenced this pull request Mar 22, 2024
* Create SECURITY.md

* Don't require license header on SECURITY.md

* Autoformat SECURITY.md

---------

Co-authored-by: Vincent Donato <vincent@streamlit.io>
zyxue pushed a commit to zyxue/streamlit that referenced this pull request Apr 16, 2024
* Create SECURITY.md

* Don't require license header on SECURITY.md

* Autoformat SECURITY.md

---------

Co-authored-by: Vincent Donato <vincent@streamlit.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants