[PR #286] [MERGED] 7.8 update #13782

Closed
opened 2026-05-17 16:12:01 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/imputnet/cobalt/pull/286
Author: @wukko
Created: 12/25/2023
Status: Merged
Merged: 12/25/2023
Merged by: @wukko

Base: currentHead: 7.8


📝 Commits (10+)

  • f500d8b better support section in about
  • dd563eb api: rework url parsing
  • 2e1eb1b api: rename hostOverrides to 'url'
  • 149c16a url: make youtu.be alias rule more lax
  • f9feaa4 tumblr: stricter subdomain parsing
  • c458423 match: light cleanup
  • 3056624 servicesConfig: set up subdomains
  • 6623605 url: return host instead of bool for success
  • 30c9652 url: typo
  • 81e68c3 processing: pass URL object instead of string

📊 Changes

27 files changed (+448 additions, -308 deletions)

View changed files

📝 package.json (+2 -1)
📝 src/config.json (+15 -13)
📝 src/core/api.js (+1 -1)
📝 src/front/cobalt.css (+38 -38)
📝 src/front/cobalt.js (+2 -2)
src/front/emoji/bubbles.svg (+30 -0)
src/front/emoji/loudspeaker.svg (+6 -0)
src/front/emoji/wrench.svg (+3 -0)
src/front/updateBanners/catroomba.webp (+0 -0)
📝 src/localization/languages/en.json (+9 -7)
📝 src/localization/languages/ru.json (+10 -8)
📝 src/localization/manager.js (+6 -1)
📝 src/modules/api.js (+17 -20)
📝 src/modules/changelog/changelog.json (+21 -11)
📝 src/modules/config.js (+9 -0)
📝 src/modules/emoji.js (+4 -1)
📝 src/modules/pageRender/elements.js (+22 -34)
📝 src/modules/pageRender/page.js (+15 -14)
src/modules/processing/hostOverrides.js (+0 -48)
📝 src/modules/processing/match.js (+12 -8)

...and 7 more files

📄 Description

fixes:

  • tlds are now parsed and validated correctly (e.g. "pinterest.co.uk" works now)
  • fixes #264 - urls with paths containing dots now work (example)
  • fixes #275 - query parameters are now parsed and handled correctly
  • subdomains and tlds are properly (& mostly strictly) validated (examples of some urls that are no longer considered valid)
  • fixes #273 - cobalt now falls back to mp3 instead of throwing an error

improvements:

  • url patterns are pre-compiled instead of being compiled for every request
  • aliases are computed in a safe manner using the URL object where possible
  • frontend no longer mangles the url when sending to api
  • api is now more tolerant to urls with whitespace around them (e.g. https://x.com/... works) - useful since pasting urls with leading/trailing spaces by accident happens quite often
  • better support section in about with a link to the status page and better sorting
  • cleaner, more compact look for the home page

(WIP)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/imputnet/cobalt/pull/286 **Author:** [@wukko](https://github.com/wukko) **Created:** 12/25/2023 **Status:** ✅ Merged **Merged:** 12/25/2023 **Merged by:** [@wukko](https://github.com/wukko) **Base:** `current` ← **Head:** `7.8` --- ### 📝 Commits (10+) - [`f500d8b`](https://github.com/imputnet/cobalt/commit/f500d8b5f95b0d66012095405131069f689394c6) better support section in about - [`dd563eb`](https://github.com/imputnet/cobalt/commit/dd563eb752f0883e295b6042db3387f802d5871f) api: rework url parsing - [`2e1eb1b`](https://github.com/imputnet/cobalt/commit/2e1eb1b864b071f976777105b758729c600f47d1) api: rename hostOverrides to 'url' - [`149c16a`](https://github.com/imputnet/cobalt/commit/149c16abbb7dbbdceaf3e1ed138608a5f55c435f) url: make youtu.be alias rule more lax - [`f9feaa4`](https://github.com/imputnet/cobalt/commit/f9feaa41ce77b14c3782ea64acc74361780f6853) tumblr: stricter subdomain parsing - [`c458423`](https://github.com/imputnet/cobalt/commit/c458423e03153bbb4d40cbb894128f8324429bd5) match: light cleanup - [`3056624`](https://github.com/imputnet/cobalt/commit/3056624b3d574c2ef8d2bcff7005cba46041c095) servicesConfig: set up subdomains - [`6623605`](https://github.com/imputnet/cobalt/commit/662360509c4c55b3431db8bbd3c82f2f5a927ba2) url: return host instead of bool for success - [`30c9652`](https://github.com/imputnet/cobalt/commit/30c9652b6e8803509cc7d563113214bb583a793b) url: typo - [`81e68c3`](https://github.com/imputnet/cobalt/commit/81e68c37f500218ce79793981678941ee97879b6) processing: pass URL object instead of string ### 📊 Changes **27 files changed** (+448 additions, -308 deletions) <details> <summary>View changed files</summary> 📝 `package.json` (+2 -1) 📝 `src/config.json` (+15 -13) 📝 `src/core/api.js` (+1 -1) 📝 `src/front/cobalt.css` (+38 -38) 📝 `src/front/cobalt.js` (+2 -2) ➕ `src/front/emoji/bubbles.svg` (+30 -0) ➕ `src/front/emoji/loudspeaker.svg` (+6 -0) ➕ `src/front/emoji/wrench.svg` (+3 -0) ➕ `src/front/updateBanners/catroomba.webp` (+0 -0) 📝 `src/localization/languages/en.json` (+9 -7) 📝 `src/localization/languages/ru.json` (+10 -8) 📝 `src/localization/manager.js` (+6 -1) 📝 `src/modules/api.js` (+17 -20) 📝 `src/modules/changelog/changelog.json` (+21 -11) 📝 `src/modules/config.js` (+9 -0) 📝 `src/modules/emoji.js` (+4 -1) 📝 `src/modules/pageRender/elements.js` (+22 -34) 📝 `src/modules/pageRender/page.js` (+15 -14) ➖ `src/modules/processing/hostOverrides.js` (+0 -48) 📝 `src/modules/processing/match.js` (+12 -8) _...and 7 more files_ </details> ### 📄 Description fixes: - tlds are now parsed and validated correctly (e.g. "pinterest.co.uk" works now) - fixes #264 - urls with paths containing dots now work ([example](https://www.instagram.com/stories/hoseobiiiiiii._.0410/3257661247380116688)) - fixes #275 - query parameters are now parsed and handled correctly - subdomains and tlds are properly (& mostly strictly) validated ([examples of some urls that are no longer considered valid](https://gist.github.com/dumbmoron/1e667f42463820fcce299b910c777dd5)) - fixes #273 - cobalt now falls back to mp3 instead of throwing an error improvements: - url patterns are pre-compiled instead of being compiled for every request - aliases are computed in a safe manner using the URL object where possible - frontend no longer mangles the url when sending to api - api is now more tolerant to urls with whitespace around them (e.g. ` https://x.com/... ` works) - useful since pasting urls with leading/trailing spaces by accident happens quite often - better support section in about with a link to the status page and better sorting - cleaner, more compact look for the home page (WIP) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-17 16:12:01 -05:00
Sign in to join this conversation.