Skip to content

Commit

Permalink
Merge pull request #11 from johnpc/allow-empty-password
Browse files Browse the repository at this point in the history
allow empty password
  • Loading branch information
johnpc authored Jan 2, 2025
2 parents 366cddf + 8e8b6aa commit 92a426b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion src/components/pages/ServerConfigurationPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,6 @@ const ServerConfigurationPage = () => {
placeholder={`Password${serverUrlOverride ? "" : " - not required if Auth Token is specified"}`}
value={password}
onChange={handlePasswordChange}
required={!authToken}
/>
</Flex>

Expand Down
2 changes: 1 addition & 1 deletion src/lib/jellyfin-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ export const getAuthenticatedJellyfinApi = async (): Promise<Api> => {
const password = getCacheValue(JELLYFIN_PASSWORD_CACHE_KEY) ?? "";
const jellyfinAuthToken = getCacheValue(JELLYFIN_AUTH_TOKEN_CACHE_KEY);

if (!serverUrl || (!(username && password) && !jellyfinAuthToken)) {
if (!serverUrl || (!username && !jellyfinAuthToken)) {
throw new Error(
"Missing credentials in localStorage. Please configure your Jellyfin connection.",
);
Expand Down

0 comments on commit 92a426b

Please sign in to comment.