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

Release 0006 #160

Merged
merged 6 commits into from
Nov 28, 2023
Merged

Release 0006 #160

merged 6 commits into from
Nov 28, 2023

Conversation

JamesPeck
Copy link
Contributor

No description provided.

Gcolon021 and others added 6 commits November 14, 2023 06:06
* [ALS-5000] Update maven compiler plugin to latest
version. Previous version has a dependency on log4j.
ALS-4957] BDC StatViz: Continuous bar should show true number when participants have only one value.
[ALS-5228] BDC Open StatViz was always obfuscated. It is now obfuscated as needed.
[ALS-5197] Fix "Other" bar not displaying in Authorized PIC-SURE
- Requirement: in order to know where to send data, we need
to associate each query with the requester's site
- Create table of site code, site name, site email domain
- When an institutional query request is made:
  - Parse the site code from the email of the requesting user
  - Add that site code to the query request
- When persisting a query object after starting a query request
  - Check for a site code in the query request
  - Add it to the query meta if it exists
- Add some extra niceness for later data transfer work
  - S3 status code
- Interface, implementations for general use, GIC
- Repos I verified this change with:
  - pic-sure-auth-microapp: master and fence branches
  - pic-sure-gic-common-frontend: Search resource, UUID gen resource
…ns (#158)

* [ALS-5356] Remove empty harmonized consent graph
- We don't want an attacker leveraging the proxy client
to make requests to 3rd party domains
- At the moment, this is just intended for docker containers in the
same network, so I just made a regex that restricts the host to
reasonable container names
@JamesPeck JamesPeck merged commit 9d9b072 into master Nov 28, 2023
6 checks passed
@JamesPeck JamesPeck deleted the release/0006 branch November 28, 2023 13:42
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.

2 participants