Skip to content

Update/multidomains 01 #924

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

rastislavcore
Copy link

This PR is based on clened version of code, bug fixes and all dependencies are up to date.
New feature is multi-domain use.

Updates multiple dependencies to their latest versions, including express, pg, bcryptjs, and others, to enhance security and performance.

Also corrects the PostgreSQL Docker Compose configuration to remove an unnecessary newline character.
Updates `express` and other dependencies to address potential security vulnerabilities and improve compatibility.

Replaces the `nanoid` package with `node:crypto`'s `randomUUID` or a custom ID generation function for creating API keys and short link IDs.
This removes a dependency and enhances security by leveraging the built-in crypto module.
Adds support for multiple domains, allowing users to create short links using domains defined in the environment configuration, in addition to user-owned domains.

Introduces a domain_name field to the links table to accommodate links created via environment-configured domains.

Updates the UI and validation logic to handle both database and environment-configured domains seamlessly.
This change removes the custom views, including header, footer, and shortener, effectively reverting to a more basic or default template structure.

It also moves Git link to the default header.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant