Closed Bender-22 closed 3 weeks ago
The recent changes involve updating the image source URLs across various JSP files in the web application. The updates switch the URL paths from an insecure HTTP endpoint to a secured HTTPS endpoint, https://effigy.im/a/
, for fetching contributor avatars. This standardizes the URLs and enhances security.
File Path | Change Summary |
---|---|
src/main/webapp/WEB-INF/jsp/admin/layout.jsp src/main/webapp/WEB-INF/jsp/analytics/... src/main/webapp/WEB-INF/jsp/content/contributor/contributor-numbers.jsp ... |
Updated image source URLs to use https://effigy.im/a/ , replacing http://62.75.236.14:3000/identicon/ and similar insecure URLs. |
Objective (Issue #38) | Addressed | Explanation |
---|---|---|
Generate identicons from ETH address | ❌ | The changes only update URLs for fetching images, not generating identicons from ETH addresses in JavaScript as required in the issue. |
The provided code changes focus on altering the image source URLs to a different domain and do not address the issue of generating identicons from ETH addresses using JavaScript.
src/main/webapp/WEB-INF/jsp/content/word/peer-reviews/pending.jsp (1)
`47-47`: **LGTM! URL change to HTTPS is secure.** The URL for contributor images has been updated to use `https://effigy.im/a/`, enhancing security and reliability.src/main/webapp/WEB-INF/jsp/content/storybook/peer-reviews/pending.jsp (1)
`62-62`: **LGTM! URL change to HTTPS is secure.** The URL for contributor images has been updated to use `https://effigy.im/a/`, enhancing security and reliability.src/main/webapp/WEB-INF/jsp/content/contributor/list.jsp (1)
`17-17`: **LGTM! URL change to HTTPS is secure.** The URL for contributor images has been updated to use `https://effigy.im/a/`, enhancing security and reliability.src/main/webapp/WEB-INF/jsp/content/contributor/contributor-summarized.jsp (1)
`10-10`: **LGTM! URL change to HTTPS is secure.** The URL for contributor images has been updated to use `https://effigy.im/a/`, enhancing security and reliability.src/main/webapp/WEB-INF/jsp/content/letter/edit.jsp (2)
`83-83`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain. --- `115-115`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain.src/main/webapp/WEB-INF/jsp/analytics/layout.jsp (1)
`80-80`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain.src/main/webapp/WEB-INF/jsp/content/contributor/contributor-numbers.jsp (3)
`51-51`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain. --- `125-125`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain. --- `178-178`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain.src/main/webapp/WEB-INF/jsp/admin/layout.jsp (1)
`85-85`: **Approve the change to use HTTPS for the image source.** The change to use `https://effigy.im/a/` enhances security by using HTTPS. Verify the reliability of the new domain.src/main/webapp/WEB-INF/jsp/content/number/edit.jsp (2)
`191-191`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain. --- `223-223`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain.src/main/webapp/WEB-INF/jsp/content/main.jsp (3)
`171-171`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain. --- `204-204`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain. --- `237-237`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain.src/main/webapp/WEB-INF/jsp/content/multimedia/image/edit.jsp (1)
`125-125`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain.src/main/webapp/WEB-INF/jsp/content/layout.jsp (1)
`144-144`: **URL update approved.** The change to the image source URL ensures secure access via HTTPS using the new domain.src/main/webapp/WEB-INF/jsp/content/storybook/edit.jsp (2)
`211-211`: **LGTM! Updated image source URL enhances security.** The change to use `https://effigy.im/a/` for the contributor avatars improves security by using HTTPS. --- `269-269`: **LGTM! Updated image source URL enhances security.** The change to use `https://effigy.im/a/` for the peer review event contributor avatars improves security by using HTTPS.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 15.03%. Comparing base (
f42424d
) to head (3662091
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
close https://github.com/elimu-ai/web3-wiki/issues/38