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

develop #847

Open
wants to merge 2,493 commits into
base: master
Choose a base branch
from
Open

develop #847

wants to merge 2,493 commits into from

Conversation

sou-chon
Copy link
Collaborator

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2020

This pull request fixes 24 alerts when merging 20aed48 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 1, 2020

This pull request fixes 24 alerts when merging 4e87744 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2020

This pull request fixes 24 alerts when merging bd6a33e into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2020

This pull request fixes 24 alerts when merging d2cecbd into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2020

This pull request fixes 24 alerts when merging a36f68c into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging 3cde989 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging 6a168cb into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging 0e1bd29 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging ce2993e into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging 5cb5e66 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request fixes 24 alerts when merging aa94aa1 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2020

This pull request fixes 24 alerts when merging a42665b into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2020

This pull request fixes 24 alerts when merging 1d924e9 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2020

This pull request fixes 24 alerts when merging 51602ec into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 7, 2020

This pull request fixes 24 alerts when merging 4520f1f into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 7, 2020

This pull request fixes 24 alerts when merging 1c097d7 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2020

This pull request fixes 24 alerts when merging 3daa64a into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Nov 20, 2020

This pull request fixes 24 alerts when merging b89c3f9 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Dec 14, 2020

This pull request fixes 24 alerts when merging 3726e6e into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Dec 27, 2020

This pull request fixes 24 alerts when merging 1e5a05c into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 15, 2021

This pull request fixes 24 alerts when merging 1b0a87d into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 15, 2021

This pull request fixes 24 alerts when merging b5c7c9c into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 17, 2021

This pull request fixes 24 alerts when merging 695e9a8 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 18, 2021

This pull request fixes 24 alerts when merging 37a5855 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 18, 2021

This pull request fixes 24 alerts when merging 41b7bed into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Feb 3, 2021

This pull request fixes 24 alerts when merging 166301b into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Feb 4, 2021

This pull request fixes 24 alerts when merging 2fe71b3 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Feb 4, 2021

This pull request fixes 24 alerts when merging 77a8995 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Feb 5, 2021

This pull request fixes 24 alerts when merging 32307dc into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Feb 5, 2021

This pull request fixes 24 alerts when merging e38b8f1 into 4ceee58 - view on LGTM.com

fixed alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Duplicate switch case
  • 1 for Unreachable statement
  • 1 for Variable not declared before use
  • 1 for Ignoring result from pure array method
  • 1 for Hard-coded credentials

fguitton and others added 26 commits January 26, 2022 22:50
@fguitton
Merge branch 'fix/webpack-server-runner-step-2' into develop
99a5e43 
# Conflicts:
#	packages/itmat-interface/package.json
#	packages/itmat-job-executor/package.json
#	yarn.lock
@fguitton
chore: Removing extraneous tsconfig.build.json files
ca3b808
@fguitton
chore: Fixing typo in profile component naming
cc0e6cf
@fguitton
chore: Cleanup dependencies
12af0ba
@fguitton
feat: Upgrade to Apollo GraphQL v3.x stack
1f8eb34
@fguitton
fix: Spread uploadTime out of tests asserts
5156123
@fguitton
Merge pull request #787 from ideafast/chore/update-apollo
2e48958 
feat: Upgrade to Apollo GraphQL v3.x stack
@fguitton
chore: v1.0.1
092ee02
@fguitton
fix: rateLimit middleware is no longer a constructor
b5c6db6
@fguitton
chore: v1.0.2
50c652e
@fguitton
chore: Add LGTM exclusion configuration
f6a8566
@fguitton
chore: Cleaning up force yarn resolutions
acefa9d
@dependabot
chore(deps-dev): bump @types/node from 17.0.12 to 17.0.13
2c71608 
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 17.0.12 to 17.0.13.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@fguitton
Merge pull request #790 from ideafast/dependabot/npm_and_yarn/develop…
2587476 
…/types/node-17.0.13

chore(deps-dev): bump @types/node from 17.0.12 to 17.0.13
@fguitton
Merge pull request #738 from ideafast/develop
8a8434f 
v1.0.2
@fguitton
Merge remote-tracking branch 'ideafast/master' into upstream/chore/me…
afbdfa5 
…rge-base-1
@fguitton
Merge commit 'b5d5658282684ff9406307606679eec3ddd511d2' into upstream…
ac478e1 
…/chore/merge-base-1
@fguitton
Merge commit '8d75dbe44afbb967a1f10933a24fc5b5131e5615' into upstream…
afcf185 
…/chore/merge-base-1
@fguitton
Merge commit 'd06d5bc1c93da07a2684b04ee6f8579772addd26' into upstream…
9d7709f 
…/chore/merge-base-1

# Conflicts:
#	packages/itmat-commons/src/permissions.ts
@fguitton
Merge commit 'f2f940c11797315db1cfb6fa1ceb83171a763950' into upstream…
adf6fac 
…/chore/merge-base-1

# Conflicts:
#	packages/itmat-commons/src/index.ts
#	packages/itmat-commons/src/permissions.ts
@fguitton
Merge commit '2bd7507b2fb2bcdd26f4668d13006d309f395056' into upstream…
99c20ee 
…/chore/merge-base-1
@fguitton
Merge commit '1108500e1810e5077ef6f4f16fd1af01b7f0c8ca' into upstream…
96c2f1c 
…/chore/merge-base-1

# Conflicts:
#	packages/itmat-interface/src/graphql/resolvers/fileResolvers.ts
#	packages/itmat-interface/src/graphql/resolvers/jobResolvers.ts
#	packages/itmat-interface/src/graphql/resolvers/studyResolvers.ts
#	packages/itmat-interface/src/rest/fileDownload.ts
#	packages/itmat-ui-react/src/components/reusable/roleControlSection/roleControlSection.tsx
@fguitton
Merge commit '99594403ee2647034a9bd661e38f9588e4dda206' into upstream…
369ff3e 
…/chore/merge-base-1
@fguitton
Merge commit '87328ece43c67ea20fece2f845dc7e29b6b0f319' into upstream…
7085cf5 
…/chore/merge-base-1

# Conflicts:
#	packages/itmat-interface/test/serverTests/job.test.ts
#	packages/itmat-interface/test/serverTests/permission.test.ts
#	packages/itmat-interface/test/serverTests/study.test.ts
@fguitton
Merge commit '1c4f808b8d5649e2aa137bab648d79fb7cd22577' into upstream…
880e6f1 
…/chore/merge-base-1

# Conflicts:
#	packages/itmat-interface/test/unitTests/permissionCore.test.ts
@fguitton
Merge branch 'upstream/chore/merge-base-1' into upstream/develop
321372a 
# Conflicts:
#	.github/workflows/99_ci.yml
#	package.json
#	packages/itmat-commons/package.json
#	packages/itmat-interface/package.json
#	packages/itmat-job-executor/package.json
#	packages/itmat-setup/package.json
#	packages/itmat-ui-react/package.json
#	yarn.lock
fguitton added a commit that referenced this pull request Apr 5, 2022
@fguitton
Merge pull request #847 from ideafast/dependabot/npm_and_yarn/develop…
4905876 
…/babel/core-7.17.5

chore(deps-dev): bump @babel/core from 7.17.4 to 7.17.5
github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 5, 2022
View reviewed changes
packages/itmat-interface/src/server/router.ts
Comment on lines +44 to +52
session({
secret: config.sessionsSecret,
saveUninitialized: false,
resave: true,
rolling: true,
cookie: {
maxAge: 2 * 60 * 60 * 1000 /* 2 hour */
}
})

Check warning

Code scanning / CodeQL

Clear text transmission of sensitive cookie

Sensitive cookie sent without enforcing SSL encryption
@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/codeql.yml:CodeQL-Build. As part of the setup process, we have scanned this repository and found 2 existing alerts. Please check the repository Security tab to see all alerts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sou-chon @wsy19961129 @nguyentb @renovate-bot @fguitton