vikunja/vikunja
6
46
Fork 75
Code Issues 152 Pull Requests 31 Packages Releases 34 Activity

Detection of https API doesn't work anymore from http frontend #2035

New Issue
Closed
opened 2023-03-29 11:04:55 +00:00 by dpschen · 4 comments
dpschen commented 2023-03-29 11:04:55 +00:00
Member
Copy Link

Description

  1. Start a local version of the frontend with only http.
  2. Try to connect to an api instance by only typing in the host e.g. try.vikunja.io.

Expected result

Frontend should try if there is an https version of the api and connect to that if it exists. Unsure because of possible security implications: it should follow 301 redirects.

Actual result

Frontend tries to connect to the host via http. This will fail for an https api, so http://try.vikunja.io will fail.

Vikunja Frontend Version

dev

Vikunja API Version

v0.20.4+40-5114f53307

Browser and version

Chromium

Can you reproduce the bug on the Vikunja demo site?

No

Screenshots

image

### Description 1) Start a local version of the frontend with only http. 2) Try to connect to an api instance by only typing in the host e.g. `try.vikunja.io`. ## Expected result Frontend should try if there is an https version of the api and connect to that if it exists. Unsure because of possible security implications: it should follow 301 redirects. ## Actual result Frontend tries to connect to the host via http. This will fail for an https api, so `http://try.vikunja.io` will fail. ### Vikunja Frontend Version dev ### Vikunja API Version v0.20.4+40-5114f53307 ### Browser and version Chromium ### Can you reproduce the bug on the Vikunja demo site? No ### Screenshots ![image](/attachments/cbdd6c03-c116-4efb-b6f2-3af060368b06)
image.png
33 KiB
dpschen added the
kind/bug
 label 2023-03-29 11:04:55 +00:00
dpschen changed title from Automatic detection of API end result doesn't work anymore from local dev to Detection of https API doesn't work anymore from http frontend 2023-03-29 11:08:11 +00:00
konrad commented 2023-03-29 11:09:00 +00:00
Owner
Copy Link

Does it work if you enter the url with the protocol?

Does it work if you enter the url with the protocol?
konrad commented 2023-03-29 11:09:34 +00:00
Owner
Copy Link

Probably a regression by vikunja/frontend#3303

Probably a regression by https://kolaente.dev/vikunja/frontend/pulls/3303
👍 1
dpschen commented 2023-03-29 11:09:43 +00:00
Author
Member
Copy Link

Does it work if you enter the url with the protocol?

Yes. But typing that in is a bit annoying.

> Does it work if you enter the url with the protocol? Yes. But typing that in is a bit annoying.
👍 1
dpschen commented 2023-03-29 20:20:21 +00:00
Author
Member
Copy Link

Related: vikunja/frontend#1019 (comment)

Related: https://kolaente.dev/vikunja/frontend/pulls/1019#issuecomment-19608
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
dependencies

   
duplicate

This issue or pull request already exists

  
help wanted

Need some help

  
invalid

Something is wrong

  
kind/bug

Something is not working

  
kind/feature

New feature

  
needs reproduction

   
question

More information is needed

  
security

   
wontfix

This won't be fixed

No Label
dependencies
duplicate
help wanted
invalid
kind/bug
kind/feature
needs reproduction
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#2035
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?