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

livechat: Cannot View Images and Attachment #28231

Open
offer1983 opened this issue Oct 29, 2018 · 10 comments
Open

livechat: Cannot View Images and Attachment #28231

offer1983 opened this issue Oct 29, 2018 · 10 comments
Assignees
Labels
Discuss Discuss, mail, chatter, livechat, attachments

Comments

@offer1983
Copy link

offer1983 commented Oct 29, 2018

livechat: Cannot View Images and Attachment

Website visitors cannot view attachments while chatting.

20181029123708

20181029124244

@Yenthe666 Yenthe666 added the Framework General frontend/backend framework issues label Oct 30, 2018
@jeanlucmongrain
Copy link

that is probably a permission issue, but I couldn't figure which one. might be related to #35037

@pedrobaeza
Copy link
Collaborator

Hi, thanks for reporting. Are you able to reproduce the problem in runbot infrastructure (http://runbot.odoo.com/runbot - click on the blue door icon of the version row - admin/admin)?

If so, please explain with all the possible detailed steps to get to the error. Not having them, it's very difficult to hunt down the problem. Please don't be shy of adding details, and even better if you provide some screenshots or videos.

If not, this can be due to a configuration change, a custom/community module you have deployed in your DB or an outdated code. Please report back if it's the case.

For keeping a clean house in this issue tracker, if you don't answer in several days, this will be closed assuming it was one of the stated things.

@pedrobaeza pedrobaeza added the Need information the reports lacks of information label Jun 19, 2020
@jeanlucmongrain
Copy link

jeanlucmongrain commented Jun 19, 2020

I just tried with the last build of 13.0 branch and I could reproduce it

screenshot

On the left side is the visitor (not logged session) and on the right side is admin user logged in.

admin user attached some random picture, the visitor gets a broken image URL.

The picture URL is /web/content/1587?download=true which 404 for visitor. But admin user image is valid, as I can download that file

As the same attachment URL is the same for both visitor and admin, it confirms my theory that it's a permission issue.

@pedrobaeza pedrobaeza removed the Need information the reports lacks of information label Jun 19, 2020
@pedrobaeza
Copy link
Collaborator

OK, thanks for the confirmation. @seb-odoo something to consider with #35037

@jeanlucmongrain
Copy link

OK, thanks for the confirmation. @seb-odoo something to consider with #35037

well, not exactly. it's not the same problem as I first suspected. as in this specific runbot instance I do see the avatar of the agent that chat with the visitor. I can't reproduce #35037

@seb-odoo
Copy link
Contributor

It's still related though, as the root cause is likely the same: being able to see a message does not guarantee that other records referenced in the message are accessible.

It's not an obvious fix, and probably not fully fixable in stable, so I will tackle these after the OWL refactoring of mail is fully complete (it was merged but a lot still has to be fixed/converted including the public livechat). Related task would be 2223236.

@seb-odoo seb-odoo self-assigned this Jun 19, 2020
@seb-odoo seb-odoo added Discuss Discuss, mail, chatter, livechat, attachments and removed Framework General frontend/backend framework issues labels Jun 19, 2020
@ska-ibees
Copy link
Contributor

@seb-odoo Could you please share any update/ETA/temorary fix for this? We badly need file sharing in public chat and this issue still persists in 14.0 as well. Thanks.

@emagdalenaC2i
Copy link

same issue here with 14.0

@seb-odoo
Copy link
Contributor

It's not done yet but it should be in v15 hopefully.

@ska-ibees
Copy link
Contributor

Thanks for the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discuss Discuss, mail, chatter, livechat, attachments
Projects
None yet
Development

No branches or pull requests

7 participants