Images with QueryString Parameters are Incorrectly Interpreted (Steemit/Condenser)

in #utopian-io6 years ago (edited)

Project Information

Expected behavior

Images that are with query string parameters should be displayed correctly.

Actual behavior

Images that are with query string parameters are ignored, and therefore displayed incorrectly or broken.

How to reproduce

I have got two images:

  1. https://uploadbeta.com/api/pictures/random/test/?key=aa
  2. https://uploadbeta.com/api/pictures/random/test/

As you can see, the first image URL has a query string,, they are two totally different images. If you add them to steemit, they will automatically prepended:

  1. https://cdn.steemitimages.com/0x0/https://uploadbeta.com/api/pictures/random/test/?key=aa
  2. https://cdn.steemitimages.com/0x0/https://uploadbeta.com/api/pictures/random/test/

However, The image with query string is shown incorrectly, in fact, it is shown as the second image.

First Image (with query string parameters )

Second Image

Please note that if you use busy.org to read this post, two images are displayed correctly

This is a critical bug which I think has been introduced lately (regression). I found this bug because I use a API to insert pictures with query parameters but suddenly it doesn't work anymore.

  • Browser/App version: Chrome v66
  • Operating system: Win 10 64-bit

GitHub Account

https://github.com/doctorlai

Sort:  

小霸王其乐无穷!!!

busy 用户,鉴定完毕 ^_^

哈哈,这里面有啥讲究吗?难不成还存在某种鄙视链?
看到了小霸王学习机,勾起了回忆……

You are doing great work for community..

Would love if you play my little contest here..

https://steemit.com/contest/@ivrmakers/spot-the-idiot-contest-3-win-0-05-sbd

& I will vote for your witness.

Thank you for contributing to something that hits so close to home! It's a rare issue that will not cause problems for most users, but is surely a major bug for those that do want to use this functionality.

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Thanks, I've just raised this in the github: https://github.com/steemit/condenser/issues/2822
BTW, the utopian-io does not automatically raise issues once it is approved?

It doesn't anymore due to Github TOS. It's technically an advertisement in other people's issues. That's why we encourage users to do it by themselves without mentioning Utopian.

I'm not sure if linking the steemit post would go against their TOS (surely not in this case because it's a steemit report, but in other cases maybe), but I would advise against it in the future. Because of Steemit & Github being markdown based you can just copypaste the content into the issue without losing clarity.

Hey @justyy
Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!

Contributing on Utopian
Learn how to contribute on our website or by watching this tutorial on Youtube.

Want to chat? Join us on Discord https://discord.gg/h52nFrV.

Vote for Utopian Witness!

This has now been resolved. Thank you for your bug report submission on GitHub.

Not resolved. Sorry. Still cannot upload images.

I think you are talking about a different issue.

Coin Marketplace

STEEM 0.16
TRX 0.12
JST 0.026
BTC 60313.79
ETH 2889.34
USDT 1.00
SBD 2.47