App icon indicating copy to clipboard operation
App copied to clipboard

[$50] "Your data is protected" link is broken

Open m-natarajan opened this issue 1 year ago β€’ 26 comments

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 9.0.24-2 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: @hannojg Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724679642375049

Action Performed:

  1. Go to staging.new.expensify.com
  2. Initiate Add Business Account flow
  3. Click Your Data Is Secure link

Expected Result:

https://community.expensify.com/discussion/5677/deep-dive-how-expensify-protects-your-information/ opens without any issue

Actual Result:

Link broken and Error 1014 displayed

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • [ ] Android: Native
  • [ ] Android: mWeb Chrome
  • [ ] iOS: Native
  • [ ] iOS: mWeb Safari
  • [x] MacOS: Chrome / Safari
  • [ ] MacOS: Desktop

Screenshots/Videos

https://github.com/user-attachments/assets/c6cd3cee-348f-40d3-985b-85f246a70ea1

Snip - CNAME Cross-User Banned Cloudflare - Google Chrome

Add any screenshot/video evidence

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0194501906e3889a0d
  • Upwork Job ID: 1828335152483406949
  • Last Price Increase: 2024-08-27
Issue OwnerCurrent Issue Owner: @fedirjh

m-natarajan avatar Aug 26 '24 20:08 m-natarajan

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

melvin-bot[bot] avatar Aug 26 '24 20:08 melvin-bot[bot]

Job added to Upwork: https://www.upwork.com/jobs/~0194501906e3889a0d

melvin-bot[bot] avatar Aug 27 '24 07:08 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh (External)

melvin-bot[bot] avatar Aug 27 '24 07:08 melvin-bot[bot]

Upwork job price has been updated to $50

melvin-bot[bot] avatar Aug 27 '24 07:08 melvin-bot[bot]

Edited by proposal-police: This proposal was edited at 2024-08-27 09:59:28 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue. "Your data is protected" link is not working.

What is the root cause of that problem?

The error persists even when the link is manually pasted into a new browser tab, indicating a problem outside of the our Expensify application.

The root cause of "Error 1014" is likely related to a CNAME Cross-User Banned issue on Cloudflare. This error typically occurs when one domain points to another via a CNAME record, and the domains are managed under different Cloudflare accounts. The issue seems to originate from the configuration of the community.expensify.com domain or its Cloudflare settings.

What changes do you think we should make in order to solve the problem?

If we want to use the same link then contact the administrators responsible for community.expensify.com. Inform them of the "Error 1014" and request that they review the CNAME configurations within their Cloudflare account.

What alternative solutions did you explore? https://github.com/Expensify/App/blob/d5cfecfc53a2abf926a1dd9be8bc72c0b373ff81/src/pages/ReimbursementAccount/BankAccountStep.tsx#L168-L173 Consider providing an alternative link for users to access the content. Replace above link with https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security

ijmalik avatar Aug 27 '24 09:08 ijmalik

Expensify/App Your proposal will be dismissed because you did not follow the proposal template.

github-actions[bot] avatar Aug 27 '24 09:08 github-actions[bot]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Link is broken when clicking Your Data Is Secure

What is the root cause of that problem?

I think we already migrated from https://community.expensify.com/discussion/5677/deep-dive-how-expensify-protects-your-information/ to https://help.expensify.com/

Because when we open the https://community.expensify.com we got redirected to https://help.expensify.com/ So we need to update the link from https://community.expensify.com to https://help.expensify.com/

What changes do you think we should make in order to solve the problem?

We need to ask Expensify internal team for the right article inside the https://help.expensify.com

And update the below code: https://github.com/Expensify/App/blob/d5cfecfc53a2abf926a1dd9be8bc72c0b373ff81/src/pages/ReimbursementAccount/BankAccountStep.tsx#L168-L172

What alternative solutions did you explore? (Optional)

NJ-2020 avatar Aug 27 '24 09:08 NJ-2020

@adelekennedy, it seems that the URL is outdated and needs to be updated. Can you confirm if this is the correct link?

  • https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security

fedirjh avatar Aug 27 '24 13:08 fedirjh

@ijmalik proposal looks good to me. We just need to confirm the correct link https://github.com/Expensify/App/issues/48031#issuecomment-2312553914

πŸŽ€ πŸ‘€ πŸŽ€ C+ reviewed

fedirjh avatar Aug 27 '24 13:08 fedirjh

Triggered auto assignment to @blimpich, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

melvin-bot[bot] avatar Aug 27 '24 13:08 melvin-bot[bot]

@fedirjh Ijmalik updated the alternative solution very similar to my solution, can you please look at the timestamps where he updated the alternative solution very similar to my solution after my proposal has been sent

Before he updated the proposal, here's his original alternative solution:

If the issue cannot be resolved on the target domain, consider providing an alternative link for users to access the content.

NJ-2020 avatar Aug 27 '24 13:08 NJ-2020

@NJ-2020 I have already checked the timestamps, and the alternative solution looked correct to me as it's a straightforward fix.

fedirjh avatar Aug 27 '24 13:08 fedirjh

@fedirjh But as you can see the alternative solution it's almost exact same, the only difference he provide the link: https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security

NJ-2020 avatar Aug 27 '24 13:08 NJ-2020

@NJ-2020 Following The contributing guidelines :

Note: Before submitting a proposal on an issue, be sure to read any other existing proposals. ALL NEW PROPOSALS MUST BE DIFFERENT FROM EXISTING PROPOSALS. The difference should be important, meaningful or considerable.

I didn’t really notice any considerable difference between the two solutions. The solution is simple: we just need to update the link.

fedirjh avatar Aug 27 '24 14:08 fedirjh

@fedirjh Yes I've read, but he didn't mention about that earlier

NJ-2020 avatar Aug 27 '24 14:08 NJ-2020

Hmmm, he mentioned in his initial proposal that we can use an alternative link. Then he provided the link for us. In any case, @blimpich will make the final decision.

fedirjh avatar Aug 27 '24 14:08 fedirjh

Ok, Thanks

NJ-2020 avatar Aug 27 '24 14:08 NJ-2020

@adelekennedy the community site is being/has been deprecated. For the OP and the expected results, we should include and update in the triage the correct link we expect it to have been replaced by. If you're unsure what that replacement link is, #expensifyhelp would be a good place to ask. πŸ‘

trjExpensify avatar Aug 28 '24 00:08 trjExpensify

@adelekennedy do you know if this would be the correct link to replace this with? https://help.expensify.com/articles/new-expensify/settings/Encryption-and-Data-Security

blimpich avatar Aug 29 '24 20:08 blimpich

@blimpich, @fedirjh, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Aug 30 '24 18:08 melvin-bot[bot]

That's the correct one @blimpich

adelekennedy avatar Sep 03 '24 04:09 adelekennedy

πŸ“£ @ijmalik You have been assigned to this job! Please apply to the Upwork job and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review πŸ§‘β€πŸ’» Once you apply to this job, your Upwork ID will be stored and you will be automatically hired for future jobs! Keep in mind: Code of Conduct | Contributing πŸ“–

melvin-bot[bot] avatar Sep 03 '24 16:09 melvin-bot[bot]

I agree with @fedirjh. Lets go with @ijmalik's proposal πŸ™‚

blimpich avatar Sep 03 '24 16:09 blimpich

@ijmalik, @blimpich, @fedirjh, @adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Sep 06 '24 18:09 melvin-bot[bot]

@ijmalik are you able to raise a PR soon?

blimpich avatar Sep 06 '24 20:09 blimpich

< @ijmalik are you able to raise a PR soon? Yes, I’ll have the PR ready by the end of today.

ijmalik avatar Sep 07 '24 00:09 ijmalik

πŸ“£ @ijmalik You have been assigned to this job! Please apply to the Upwork job and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review πŸ§‘β€πŸ’» Once you apply to this job, your Upwork ID will be stored and you will be automatically hired for future jobs! Keep in mind: Code of Conduct | Contributing πŸ“–

Hi @blimpich ,

I have applied for the Upwork job, but it seems my status hasn’t been updated here yet. Please let me know if any further action is required from my side.

ijmalik avatar Sep 11 '24 11:09 ijmalik

I don't think you need to do anything else. @adelekennedy are you able to double check that @ijmalik has done all the steps they need to do in order to get paid via Upwork? I believe they have.

blimpich avatar Sep 11 '24 16:09 blimpich

sorry! This totally slipped off my radar - I've hired you in Upwork @ijmalik

adelekennedy avatar Sep 25 '24 16:09 adelekennedy

This was also deployed to production, so @adelekennedy we should be ready to pay out I believe.

blimpich avatar Sep 25 '24 17:09 blimpich