Skip to content
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

Can't add 'data:image' URL for search engine IconURL #96

Closed
jcpearson opened this issue Jul 8, 2019 · 2 comments
Closed

Can't add 'data:image' URL for search engine IconURL #96

jcpearson opened this issue Jul 8, 2019 · 2 comments
Assignees
Labels
bug Something isn't working released
Milestone

Comments

@jcpearson
Copy link

Describe the bug

The 'Icon URL' input box gives 'invalid URL' when entering an inline encoded image and doesn't add the IconURL to the output policies.json

The 'URL' I'm using is:

data:image/png;base64,iVBORw0KGgo...RK5CYII=

(string shortened for brevity)

Manually adding such an IconURL string to the generated policies.json works fine

Version information:

  • Firefox version: Firefox ESR 68.0
  • Enterprise Policy Generator version 4.4.0

Additional context

Interestingly, the input text box for the 'URL for searches' will accept a 'data:image/...' URL without complaining ...

Looking at the policy generator code (js/core/policies.js), it has:

 {
   name : 'URLTemplate',
   label : browser.i18n.getMessage('policy_description_SearchEngines_Add_URL'),
   mandatory : true,
   type : 'string'
 },
 {
   name : 'IconURL',
   label : browser.i18n.getMessage('policy_description_SearchEngines_Add_IconURL'),
   mandatory : false,
   type : 'url'
 },

Maybe the 'type' for 'IconURL' should be 'string' ?

@jcpearson jcpearson added the bug Something isn't working label Jul 8, 2019
@cadeyrn
Copy link
Owner

cadeyrn commented Jul 8, 2019

Maybe the 'type' for 'IconURL' should be 'string' ?

Sounds like a good plan. Will be changed in Enterprise Policy Generator 5.0. There is no release date for version 5.0 yet* but it will be sometime in the next few weeks.

Thank you for checking that manually adding such an IconURL works.

*) The current due date of the 5.0 milestone on GitHub does not reflect the current plannings. It will be released as soon as it's done and the due date of the 5.0 milestone can be understood as "not later than".

@cadeyrn
Copy link
Owner

cadeyrn commented Oct 10, 2019

@jcpearson: Enterprise Policy Generator 5.0 contains the fix and was released a few minutes ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

No branches or pull requests

2 participants