Repository navigation
fix: 🐛 上流CodiMD 2.5.0を取り込み画像アップロードのTLSエラーを修復 #13
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
Draft
Kentaro1043
wants to merge
138
commits into
traP
Choose a base branch
from
fix/codimd-2.5.0-image-upload
base: traP
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
138 commits
Select commit
Hold shift + click to select a range
2453b2a
set lang attr on <html> according to user's choice
binotaliu bdd2de7
set default lang attr for note to 'en'
binotaliu 9b899c7
escape attribute value for lang
binotaliu 3df0903
Convert "include" directives to functions
0aff524
Make permission labels translatable.
66fdf7b
Make permission and status translatable
dec2f98
Make "Expand all" translatable
ffac645
Remove the semicolons
ac43db8
Fix matchInContainer false positives
tamo 0ec949d
Add oauth2 authorization
4002306
Update public/js/index.js
tamo ad1bfa6
Update public/js/index.js
tamo 9343d32
Merge branch 'patch-1' of https://github.com/tamo/codimd into patch-1
5e70965
fix: handle when request url has no valid referer
jackycute db5fc50
fix: avoid append zero suffix on exporting user data
jackycute 3af5408
Merge pull request #1680 from hackmdio/bugfix/fix-export-user-data-fi…
a60814billy 54f6e55
Merge pull request #1679 from hackmdio/bugfix/fix-no-referrer-for-ret…
a60814billy a41c4db
fix: S3 client config passing for image upload
jackycute e9293b0
feat: add TeX mhchem extensions for MathJax
jackycute a913c61
feat: bump package-lock codimd version
jackycute b6837f1
feat: upgrade flowchart.js to version 1.15.0
jackycute 599fd41
Merge pull request #1683 from hackmdio/bugfix/fix-image-upload-s3-config
a60814billy b27f147
Merge pull request #1684 from hackmdio/feature/support-mhchem-mathjax
a60814billy 8c20d9f
Merge pull request #1685 from hackmdio/feature/upgrade-flowchart.js-1…
a60814billy 2338113
chore: add dev container
a60814billy 34f502c
Add test and build actions
Yukaii 71dce28
Add doctoc check
Yukaii 2432397
Fix doctoc workflow
Yukaii dcfdb01
Remove travis CI config
Yukaii b6d5db3
fix: should not clear guest history when guest pin note
a60814billy 282fcab
Merge pull request #1694 from hackmdio/chore/github-actions
a60814billy 0280a2e
feat: implement CSRF token in export user data
a60814billy b8203ae
fix: profile dropdown should not set active class
a60814billy 6966881
fix: strip html tags for gist id to avoid stored XSS on showing error…
jackycute 2eefe77
fix: strip html tags for gist file, gist line, gist highlight line, g…
jackycute e75e28c
feat: upgrade mermaid to version 8.10.1
jackycute 16a1e82
feat: upgrade mermaid to version 8.10.2
jackycute 82b7800
Merge pull request #1691 from hackmdio/bugfix/fix-gist-id-stored-XSS
a60814billy e623b72
Merge pull request #1690 from hackmdio/feature/upgrade-mermaid-8.10.1
a60814billy 1bdedf1
Merge pull request #1688 from hackmdio/chore/dev-container
jackycute 9c280b0
Add support for arm64 build
YadominJinta 6b8950d
Merge pull request #1481 from ghost/lang-attr
jackycute 4b78202
Merge pull request #1605 from tamo/patch-1
jackycute 620197f
Merge pull request #1580 from tamo/master
a60814billy e7a5ea8
Merge pull request #1587 from tamo/more-i18n
a60814billy 3b1e270
Merge pull request #1701 from YadominJinta/develop
a60814billy b2e2743
fix: may referernce out of bound index in clearDuplicatedHistory
a60814billy 0d1cd1d
feat: add organizations whitelist to GitHub OAuth
jakubgs 80fc6bc
chore: upgrade codemirror to 5.61.3
Yukaii d627d9f
Documentation - add Music section and move abc abd fretboard to this …
brunetton 5db11d2
Merge pull request #1706 from hackmdio/fix/incorrect-array-index-refe…
Yukaii e45194e
Merge pull request #1695 from hackmdio/feat/csrf-export-user-data
Yukaii c013c22
chore: bump meta-marked to 0.5.0
Yukaii e1ea214
chore: remove unused uglifyjs-webpack-plugin dep
Yukaii 08592f8
feat: deprecate sequelize.import usage
Yukaii 5b564c6
Merge pull request #1724 from hackmdio/refactor/sequelize-import
Yukaii bfda919
chore: upgrade codemirror to 5.63.2
Yukaii 32f3246
Merge pull request #1716 from hackmdio/chore/upgrade-codemirror
Yukaii ded6b58
Merge pull request #1723 from hackmdio/chore/remove-uglifyjs
Yukaii 54e5b93
Update README.md to remove IE from supporting list
jackycute adb0dd5
Fix fence class array initialization
V1ncNet 212d8b3
Update de.json
be3b2a9
Merge pull request #1741 from rubstudent/develop
a60814billy ce1b1cc
doc(release-note): Releaes 2.4.1
a60814billy 07cfb29
🐛 [fix] modify replacement rule for disqus short-name
chenxuanzzy 1a38aaa
s3 support multiple cloud providers: https://github.com/hackmdio/codi…
blademainer 7b1e4e3
s3 support multiple cloud providers: https://github.com/hackmdio/codi…
blademainer e6d2b7a
Upgrade Node.js version
inductor b55bf97
doc(release-note): change release version to 2.4.2
a60814billy 5d84066
Merge pull request #1742 from hackmdio/develop
a60814billy 03f8649
upgrading pg to 8.8.0 to support new scram-sha-256 authentication
phntom 5d72fae
fix: restrict export type and use sandbox to prevent potential attack
galaxian85 9df1c34
fix: sanitize vimeo video id to prevent XSS
galaxian85 a7564e7
Merge pull request #1697 from hackmdio/fix/guest-history-pin
jackycute 3ab0b7e
Merge pull request #1762 from blademainer/develop
jackycute 21345ed
Merge pull request #1715 from brunetton/develop
jackycute a2565bd
Merge pull request #1739 from V1ncNet/hotfix/fence-class
jackycute 8b29d05
Merge pull request #1729 from hackmdio/feature/readme-remove-IE-support
a60814billy 8efc2d0
chore: update formidable to 2.1.1
galaxian85 1ab2a36
fix: handle error when check is image valid
galaxian85 30fe18d
Merge pull request #1792 from galaxian85/bugfix/potential-xss-vulnera…
jackycute 19f494d
Merge pull request #1789 from galaxian85/bugfix/invalid-filename-caus…
a60814billy 6a5da1e
Typos + Better translation for "Externals"
eyssette 3a3e892
Merge pull request #1722 from hackmdio/chore/upgrade-meta-marked-to-0…
jackycute c78bb69
Merge pull request #1793 from eyssette/patch-2
jackycute ee47c6c
fix(buildpacks): replace custom buildpack with APT buildpack
EtienneM 6ff5a3a
feat: Migrate to gtag and support GA4
assanges 3ea10a6
Update minimum required node.js version to v12
PeterDaveHello 500fe78
Update npm package dependencies
PeterDaveHello 342ff2b
Merge pull request #1797 from EtienneM/develop
jackycute b09a472
Merge pull request #1790 from galaxian85/bugfix/pandoc-security-issue
jackycute 52bfb9b
Merge pull request #1799 from PeterDaveHello/updateDependenciesBaseline
Yukaii 08e2dcd
Merge pull request #1798 from assanges/feature/support-ga4
jackycute 4399457
reword japanese
AQ-masatoshi-yamaguchi 75d830d
Merge pull request #1802 from AQ-masatoshi-yamaguchi/reword-japanese
jackycute 20e71a3
fix: use encoded note id to update history
bbtfr 298db32
Merge pull request #1804 from bbtfr/fix-note-id
jackycute 8af3638
Merge pull request #1784 from phntom/develop
jackycute a3a789b
Merge pull request #1710 from status-im/gh-oauth-orgs
jackycute f1ed885
Merge branch 'develop' into update_node
jackycute ae38fd3
Merge pull request #1767 from inductor/update_node
jackycute be1f4cf
Merge pull request #1626 from joachimmathes/oauth2_authorization
jackycute 289db3d
Merge pull request #1750 from chenxuanzzy/bugfix/fix-regex
jackycute 09e7dfe
feat: update package-lock.json
jackycute 03e456f
Revert "feat: update package-lock.json"
jackycute e2ecaa8
feat: update package-lock.json
jackycute dd4c236
feat: update package-lock.json
jackycute 5ce1537
fix: js standard code styles
jackycute c97e224
fix: jsonlint code styles
jackycute b5b95df
fix: hide exportNoteData form in history page nav
jackycute c4f607f
Merge pull request #1808 from hackmdio/bugfix/fix-history-nav-ui
jackycute b6078a6
Update both Traditional and Simplified Chinese locales
PeterDaveHello 192698b
Update npm dependencies with `npm audit fix`, except `markmap-lib`
PeterDaveHello 976bca3
Update npm dependency markmap-lib to ^0.9.3
PeterDaveHello 3f7d202
Update npm dependencies with `npm audit fix` after markmap-lib update
PeterDaveHello 8efa75e
Update node.js version in .nvmrc
PeterDaveHello b6eba05
Merge pull request #1816 from PeterDaveHello/node.js-14
jackycute 11cd200
fix: sanitize pdf url to prevent xss
EastSun5566 2a977bc
fix: markmap api changes in 0.9.3
Yukaii 6d95fd1
Merge pull request #1817 from PeterDaveHello/updateDependenciesBaseline
Yukaii dca7f8c
Merge pull request #1832 from hackmdio/bugfix/sanitize-url-to-prevent…
Yukaii 070808b
Merge pull request #1815 from PeterDaveHello/updateLocales
Yukaii 027a4b4
fix the uploadimage form
hcyuser 96a589d
add newline here to pass the standard lint.
hcyuser 1b1ff89
fix: formidable option API changes
Yukaii f58c669
Merge pull request #1836 from hackmdio/bugfix/uploadimage-form
stanley2058 c436af1
fix: logout api should add callback
Yukaii 9790586
Merge pull request #1837 from hackmdio/bugfix/logout-callback
stanley2058 f8b0301
doc(release-note): release 2.5.0
stanley2058 9b8a2de
Merge branch 'master' into release/2.5.0
stanley2058 86bf44f
doc(release-note): update release code name
stanley2058 afe49f4
doc(release-note): update release note
stanley2058 78e6663
Merge pull request #1835 from hackmdio/release/2.5.0
jackycute 2a45b58
chore: ⬆️ 上流CodiMD 2.5.0をマージ
Kentaro1043 6c706a3
style: 🎨 traP独自コードをStandard規約に合わせる
Kentaro1043 55401c2
fix: 🔧 CIのActionsをSHA固定し実行権限を制限
Kentaro1043 e10fb8e
fix: 📌 CIのdoctocを既存ロックと同じ版に固定
Kentaro1043 47508da
fix: 🔒 イメージ作成CIもActions固定と権限制限を適用
Kentaro1043 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,2 @@ | ||
| https://github.com/alex88/heroku-buildpack-vips | ||
| https://github.com/Scalingo/apt-buildpack | ||
| https://github.com/Scalingo/nodejs-buildpack |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| # [Choice] Node.js version: 16, 14 | ||
| ARG VARIANT=14-buster | ||
| FROM mcr.microsoft.com/vscode/devcontainers/javascript-node:0-${VARIANT} | ||
|
|
||
| # [Optional] Uncomment this section to install additional OS packages. | ||
| # RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \ | ||
| # && apt-get -y install --no-install-recommends <your-package-list-here> | ||
|
|
||
| # [Optional] Uncomment if you want to install an additional version of node using nvm | ||
| # ARG EXTRA_NODE_VERSION=10 | ||
| # RUN su node -c "source /usr/local/share/nvm/nvm.sh && nvm install ${EXTRA_NODE_VERSION}" | ||
|
|
||
| # [Optional] Uncomment if you want to install more global node modules | ||
| RUN su node -c "npm install -g npm@6" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| { | ||
| "name": "CodiMD", | ||
| "dockerComposeFile": "docker-compose.yml", | ||
| "service": "app", | ||
| "workspaceFolder": "/workspace", | ||
|
|
||
| // Set *default* container specific settings.json values on container create. | ||
| "settings": { | ||
| "terminal.integrated.shell.linux": "/bin/zsh", | ||
| "sqltools.connections": [{ | ||
| "name": "Container Database", | ||
| "driver": "PostgreSQL", | ||
| "previewLimit": 50, | ||
| "server": "localhost", | ||
| "port": 5432, | ||
| "database": "codimd", | ||
| "username": "codimd", | ||
| "password": "codimd" | ||
| }], | ||
| }, | ||
|
|
||
| // Add the IDs of extensions you want installed when the container is created. | ||
| "extensions": [ | ||
| "dbaeumer.vscode-eslint", | ||
| "visualstudioexptteam.vscodeintellicode", | ||
| "christian-kohler.path-intellisense", | ||
| "standard.vscode-standard", | ||
| "mtxr.sqltools", | ||
| "mtxr.sqltools-driver-pg", | ||
| "eamodio.gitlens", | ||
| "codestream.codestream", | ||
| "github.vscode-pull-request-github", | ||
| "cschleiden.vscode-github-actions", | ||
| "hbenl.vscode-mocha-test-adapter", | ||
| "hbenl.vscode-test-explorer" | ||
| ], | ||
|
|
||
| // Use 'forwardPorts' to make a list of ports inside the container available locally. | ||
| // "forwardPorts": [], | ||
|
|
||
| "portsAttributes": { | ||
| "3000": { | ||
| "label": "CodiMD server", | ||
| "onAutoForward": "notify" | ||
| }, | ||
| "5432": { | ||
| "label": "PostgreSQL", | ||
| "onAutoForward": "notify" | ||
| } | ||
| }, | ||
|
|
||
| // Use 'postCreateCommand' to run commands after the container is created. | ||
| // "postCreateCommand": "yarn install", | ||
| "postCreateCommand": "sudo chown -R node:node node_modules && /workspace/bin/setup", | ||
|
|
||
| // Comment out connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root. | ||
| "remoteUser": "node" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| version: '3' | ||
|
|
||
| services: | ||
| app: | ||
| build: | ||
| context: .. | ||
| dockerfile: .devcontainer/Dockerfile | ||
| args: | ||
| VARIANT: 14-buster | ||
| environment: | ||
| - CMD_DB_URL=postgres://codimd:codimd@localhost/codimd | ||
| - CMD_USECDN=false | ||
| volumes: | ||
| - ..:/workspace:cached | ||
| - node_modules:/workspace/node_modules:cached | ||
|
|
||
| # Overrides default command so things don't shut down after the process ends. | ||
| command: sleep infinity | ||
|
|
||
| # Runs app on the same network as the database container, allows "forwardPorts" in devcontainer.json function. | ||
| network_mode: service:db | ||
|
|
||
| # Runs app on the same network as the database container, allows "forwardPorts" in devcontainer.json function. | ||
|
|
||
| # Uncomment the next line to use a non-root user for all processes. | ||
| # user: vscode | ||
|
|
||
| # Use "forwardPorts" in **devcontainer.json** to forward an app port locally. | ||
| # (Adding the "ports" property to this file will not forward from a Codespace.) | ||
|
|
||
| db: | ||
| image: postgres:12.7-alpine | ||
| restart: unless-stopped | ||
| volumes: | ||
| - postgres-data:/var/lib/postgresql/data | ||
| environment: | ||
| - POSTGRES_USER=codimd | ||
| - POSTGRES_PASSWORD=codimd | ||
| - POSTGRES_DB=codimd | ||
|
|
||
| # Add "forwardPorts": ["5432"] to **devcontainer.json** to forward PostgreSQL locally. | ||
| # (Adding the "ports" property to this file will not forward from a Codespace.) | ||
|
|
||
| volumes: | ||
| node_modules: | ||
| postgres-data: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| name: 'Test and Build' | ||
|
|
||
| on: | ||
| push: | ||
| pull_request: | ||
| workflow_dispatch: | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| test-and-build: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4 | ||
|
|
||
| - name: Reconfigure git to use HTTP authentication | ||
| run: > | ||
| git config --global url."https://github.com/".insteadOf | ||
| ssh://git@github.com/ | ||
|
|
||
| - uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4 | ||
| with: | ||
| path: ~/.npm | ||
| key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-node- | ||
|
|
||
| - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4 | ||
| name: Use Node.js ${{ matrix.node-version }} | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
| check-latest: true | ||
|
|
||
| - run: npm ci | ||
| - run: npm run test:ci | ||
| - run: npm run build | ||
|
|
||
| doctoc: | ||
| runs-on: ubuntu-latest | ||
| if: github.ref == 'refs/heads/master' || github.event.pull_request | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4 | ||
| - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4 | ||
| name: Use Node.js 14 | ||
| with: | ||
| node-version: 14 | ||
| check-latest: true | ||
| - name: Install doctoc-check | ||
| run: | | ||
| npm install -g doctoc@1.4.0 | ||
| cp README.md README.md.orig | ||
| npm run doctoc | ||
| diff -q README.md README.md.orig | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| v10.20.1 | ||
| v16.20.2 |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| libvips-dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.