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.

Sign up for GitHub

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

Jump to bottom

OpenRecords v3.5 - Main #558

Open
wants to merge 55 commits into
base: main
Choose a base branch
from
Open

OpenRecords v3.5 - Main #558

wants to merge 55 commits into from

Conversation

johnyu95
Copy link
Contributor

No description provided.

zgary and others added 30 commits November 10, 2020 18:41
@zgary
Re-implement server-side session and session timeout
068cfa1
@zgary
Add Clear-Site-Data header on logout
453626b
@zgary
Install pyotp and qrious; add MFA table; WIP views
7ab2419
@zgary
Create MFA blueprint and implement MFA backend functionality
9dbc957
@joelbcastillo
Merge pull request #535 from CityOfNewYork/g-zhou-openrecords-session…
f744108 
…-management
@zgary
Install pyotp and qrious; add MFA table; WIP views
6b7d054
@zgary
Create MFA blueprint and implement MFA backend functionality
436a08c
@zgary
Update MFA secret column to LargeBinary datatype
130e019
@zgary
Rebase fix
2963535
@zgary
Remove old migration
6b3b4a6
@zgary
Use file for fernet key and create utils file for fernet methods
3501805
@zgary
Update python dependencies
cf89430
@zgary
Add button to MFA manage page and add language to MFA register and ve…
81a0698 
…rify pages
@zgary
Update Pipfile
f4f3c67
@joelbcastillo
Gunicorn in pipfile; update gunicorn_config.py
f934244
@joelbcastillo
Merge pull request #538 from CityOfNewYork/g-zhou-openrecords-mfa
213e569
@zgary
Fix bugs in mfa register and remove
ac7b286
@zgary
Update celery_config and .env.example
fa10535
@zgary
Changes per PR comments
2a5add9
@zgary
Properly set default value for CELERY_CLEAR_EXPIRED_SESSION_IDS_INTERVAL
a8ce394
@zgary
Merge pull request #539 from CityOfNewYork/g-zhou-fix-mfa-bugs
b790ad9 
Fix MFA Bugs and Update .env Variables
@johnyu95
Updated specific agency instructions for DOT
476a236
@johnyu95
Update python pip packages to latest versions
26816f8
@johnyu95
Fixed bugs from deprecated functions and code clean up
ba036b1
@johnyu95
Fixed custom request forms bug for radios with no selected value
b564dc5
@johnyu95
Updated Pipfile
305c3b6
@johnyu95
Merge pull request #553 from CityOfNewYork/johnyu95-update-packages
26d9fc7 
Update Packages
@johnyu95
Added support for Azure storage on file uploads WIP
d7796e1
@johnyu95
Fixed file response uploads
65cee08
@johnyu95
File response edit and delete for azure storage
b89f119
@johnyu95 johnyu95 requested a review from zgary November 23, 2022 16:00
@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2022

This pull request introduces 7 alerts and fixes 2 when merging f8271a1 into 66a905e - view on LGTM.com

new alerts:

  • 4 for Unused import
  • 2 for Wrong name for an argument in a call
  • 1 for Unused local variable

fixed alerts:

  • 1 for Unused import
  • 1 for Uncontrolled data used in path expression

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2022

This pull request introduces 7 alerts and fixes 2 when merging f54420a into 66a905e - view on LGTM.com

new alerts:

  • 4 for Unused import
  • 2 for Wrong name for an argument in a call
  • 1 for Unused local variable

fixed alerts:

  • 1 for Unused import
  • 1 for Uncontrolled data used in path expression

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@johnyu95 johnyu95 mentioned this pull request Dec 15, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zgary zgary Awaiting requested review from zgary

Assignees

@zgary zgary

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@johnyu95 @zgary @joelbcastillo