Skip to content

Conversation

@stefan0xC
Copy link
Contributor

The new web-vault web-v2025.6.1 has done away with a feature flag that requires some changes in the Vaultwarden backend that we have not implemented yet. I.e. deleting entries would not be possible with newer clients ( dani-garcia/vaultwarden#5996). So I would wait for a new Vaultwarden release before updating to the newest web-vault.

@stefan0xC stefan0xC changed the title update web-vault to v2025.6.1 update web-vault to v2025.6.2 Jul 2, 2025
@tessus
Copy link
Contributor

tessus commented Jul 6, 2025

@stefan0xC I have noticed that in 2025.6.0, a few Send options are missing. e.g. expiration date, deactivate send, copy to clipboard.

I haven't been able to test 2025.6.2 yet. Has this been fixed?

Btw, you first comment mentions that vaultwarden has to be updated. The delete fix was already added, thus I was wondering, whether something else was missing from vw.

@stefan0xC
Copy link
Contributor Author

stefan0xC commented Jul 6, 2025

As far as I've looked into it this was intentionally added in web-v2025.3.0, cf. bitwarden/clients@bfbad99 so not sure what kind of fix you expect.

image

And my comment has not been updated since dani-garcia/vaultwarden#6004 has been added. So it might be out of date. (I have not yet checked if there are further changes required.)

@tessus
Copy link
Contributor

tessus commented Jul 6, 2025

Thanks for the reply. I am not quite sure what you mean by "was added". Removing options was added?

In my client on macOS, there are 3 more options and I remember them being available on web as well:

image

@stefan0xC
Copy link
Contributor Author

Have you looked at the commit I've linked? That change was added in web-v2025.3.0 that means it was already a part of that release. Not sure what explanation you want from me. It's not an error but seems intentional to me that this new dialog does not have the same options. If you think they are missing you should file a bug report upstream because we just make small changes to the web-vault.

@tessus
Copy link
Contributor

tessus commented Jul 6, 2025

I am not saying that this is your fault. You clearly misunderstood my question:

I don't have a bitwarden account, thus I do not know what the Send dialog looks like now. I also said that I haven't tested 2025.6.2 yet. I thought the missing options were the result of a bug in the bitwarden web client, since these options are available on the desktop client.
Since you have seen (and most likely tested the 2025.6.2 web client), I thought that you would know, whether these options were re-added as part of a fix. (A fix in bw, not vw.)

As to your question, if I saw the commit you linked. Yes, I did. However, the bw commit messages and info in PRs are often useless, because they don't explain anything. The commit you sent mentioned that the send dialog was replaced. Well, great, but they didn't say that they removed 3 options and why. That's anyone's guess really.

I'll create a bw account and open an issue at bw.

@stefan0xC
Copy link
Contributor Author

You clearly misunderstood my question

Why do you think I misunderstood your question? Which question are you referring to? Your initial question or the one where you were being intentionally obtuse?

I thought the missing options were the result of a bug in the bitwarden web client, since these options are available on the desktop client.

And I answered that this was probably done intentionally, so there's no confusion on my part about your question. I'm not sure what you were hoping by questioning the semantics of my answer. I've referenced the commit where this was changed and heavily implied that this was not fixed because it does not look like a bug to me.

I thought that you would know, whether these options were re-added as part of a fix

Like I said, this is not the case. btw: the screenshot was from the v2025.6.2 release, which you could have also asked if that was not clear.

@stefan0xC stefan0xC force-pushed the update-to-v2025.6.1 branch from bd68714 to 612c54c Compare July 9, 2025 15:43
@stefan0xC stefan0xC marked this pull request as ready for review July 9, 2025 15:44
@BlackDex BlackDex merged commit 9841f9a into dani-garcia:master Jul 9, 2025
1 check passed
@stefan0xC stefan0xC deleted the update-to-v2025.6.1 branch July 9, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants