Could you also allow localhost:${port} in DOMAIN_REGEX? about js-core HOT 5 CLOSED

Hey @shuuuuun and thanks for opening this suggestion. We've received similar questions to this in the past but since imgix does not support locally-served images, a change like this would be considered out of scope for a library like this. I would be curious to hear more about your use case and how this feature helps to support it. Thanks!

from js-core.

@sherwinski Thanks for your reply!

My use case is as follows.

• I am using the ix-img tag from vue-imgix.
  https://github.com/imgix/vue#basic-use-case
  https://github.com/imgix/vue/blob/v2.9.0/src/plugins/vue-imgix/ix-img.tsx
• In some places I am using buildUrlObject function manually to use srcset.
  https://github.com/imgix/vue#buildurlobject
  https://github.com/imgix/vue/blob/v2.9.0/src/plugins/vue-imgix/vue-imgix.ts#L47-L70
• They are internally using ImgixClient._buildURL. (when protocol is included)
  https://github.com/imgix/vue/blob/v2.9.0/src/plugins/vue-imgix/vue-imgix.ts#L76-L83
• The URLs specified for src or function are obtained from the server via API.
  • The API response already contains origin(protocol + host).
  • The imgix domain is used in the production environment, and localhost in the local development environment.
• While still using the ix-img tag, I want to be able to reference images on localhost in the local development environment.
  • It is complicated to use different img tag and ix-img tag depending on the environment or domain.

I am using vue-imgix in my case, but I think this issue is not limited to that.
I think it always happens when you get an image URL including origin via API and specify it to src through this library.

If there is a way to work around this without changing the library, I would be interested to know.

Thank you.

from js-core.

Hey @shuuuuun, thank you for providing more details and apologies for the delayed follow-up here. To answer each of your questions:

1. There isn't an out-of-the-box solution to this issue offered in this library currently. Unfortunately, switching between img and ix-img seems to be the best work around at this point.
2. We have received similar requests in the past to what you are asking for here. From our standpoint, these libraries are primarily focused on serving imgix URLs/images; therefore supporting functionality aside from that feels out of scope from that focus.

Having said that, I will bring this issue up in our next team meeting and see what others think as well. I will update this thread once I have more information.

from js-core.

Hi @shuuuuun,

After discussing it further with the team, we've ultimately decided not to support this request. The reason being that this library is meant to facilitate the reliable construction of URLs tied to the imgix rendering service. By expanding the validation rules, we are losing the ability to guarantee that reliability. And while we generally try to leave wiggle room for developer convenience, doing so in this case to support images not served through imgix feels a bit too out of scope for this project. Apologies and hope you can understand our position.

With that said, I think there is another workaround that you could considering using in your case. Instead of using localhost in your dev environment, you can use dummy domain such as sherwinski.imgix.local:

import URLBuilder from '@imgix/js-core';

let ub = new URLBuilder({
  domain: process.env.NODE_ENV === 'production' ? 'sherwinski.imgix.net' : 'sherwinski.imgix.local',
});

Then edit your local hosts file so that sherwinski.imgix.local resolves to localhost:

# /etc/hosts
127.0.0.1 sherwinski.imgix.local

Hopefully this helps you. I am going to close this issue for now but please feel free to comment back if any other questions come up.

Edit: Assuming you want to specify the port, you will likely need to set up a reverse proxy as the hosts files only handles hostnames.

from js-core.

Thank you for your consideration.
I understand your decision.

In my case, the workaround of switching img and ix-img seems to be a good one.
Editing the local hosts file would be a hassle to share the procedure with other developers, especially if a reverse proxy is required.

Thanks for providing a good service and library.

from js-core.