Skip to content

Commit

Permalink
chore: Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
hexxone committed May 7, 2024
1 parent fb50755 commit 27acaad
Show file tree
Hide file tree
Showing 14 changed files with 512 additions and 530 deletions.
6 changes: 3 additions & 3 deletions HttpsReverseProxy/HttpsReverseProxy.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,11 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Yarp.ReverseProxy" Version="2.1.0" />
<PackageReference Include="Yarp.ReverseProxy" Version="2.1.0"/>
</ItemGroup>

<Target Name="PreBuild" BeforeTargets="PreBuildEvent">
<Message Importance="high" Text="&#xA; Building Plugin...&#xA; " />
<Exec Command="dotnet build ../TeleJelly/TeleJelly.csproj" />
<Message Importance="high" Text="&#xA; Building Plugin...&#xA; "/>
<Exec Command="dotnet build ../TeleJelly/TeleJelly.csproj"/>
</Target>
</Project>
38 changes: 19 additions & 19 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,23 +41,23 @@ Created from [jellyfin-plugin-template](https://github.com/jellyfin/jellyfin-plu
## Features

- SSO Login page (at `/sso/Telegram`)
- styled similar to the regular login page
- responsive / mobile capable
- shows a "Back to Normal Login" button
- shows the Telegram Login Widget
- checks the Telegram Auth data against the backend
- if data is invalid -> show error message
- if data is valid -> takes the Jellyfin Auth Response and authenticates the user
- loading animation
- supports Custom CSS
- styled similar to the regular login page
- responsive / mobile capable
- shows a "Back to Normal Login" button
- shows the Telegram Login Widget
- checks the Telegram Auth data against the backend
- if data is invalid -> show error message
- if data is valid -> takes the Jellyfin Auth Response and authenticates the user
- loading animation
- supports Custom CSS

- Config page (reachable via Jellyfin Plugin Page)
- requires setting the Telegram Bot Token
- allows setting a List of Administrator Telegram Usernames (get full Access)
- allows forcing an external Protocol Scheme (for reverse proxies like Traefik)
- allows Creating/Editing/Deleting a "virtual" management Group
- Grants access to all OR specific Libraries for non-Administrators.
- _Note: A user needs to be Admin OR part of at least ONE Group to Log-in._
- requires setting the Telegram Bot Token
- allows setting a List of Administrator Telegram Usernames (get full Access)
- allows forcing an external Protocol Scheme (for reverse proxies like Traefik)
- allows Creating/Editing/Deleting a "virtual" management Group
- Grants access to all OR specific Libraries for non-Administrators.
- _Note: A user needs to be Admin OR part of at least ONE Group to Log-in._

## Requirements

Expand Down Expand Up @@ -105,16 +105,16 @@ E.g.: `[Telegram-Login](https://jelly.fin/sso/telegram)` or screenshots below.
## Known issues

- The `Sign in with Telegram` button will sometimes get hidden by Browser Plugins like "I don't like Cookies" or "UBlock Origin".
Try disabling these on your Jellyfin domain and inform your users.
Try disabling these on your Jellyfin domain and inform your users.

- If a User's profile picture fails to download even though the url is given (err 404), he has probably set it to private.
In this case, the plugin will fall back to its default icon.
In this case, the plugin will fall back to its default icon.

- If a User were to change/sell his Username, a random person would possibly be able to use this Service.
However, having Names over ID's is much more convenient for Management.
However, having Names over ID's is much more convenient for Management.

- If your server is publicly reachable, make sure to take care of rate limiting with your reverse proxy,
otherwise adversaries might be able to lag the system.
otherwise adversaries might be able to lag the system.

## Demo Video

Expand Down
76 changes: 38 additions & 38 deletions TeleJelly/Assets/Config/config.css
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@
width: 100%;
}

.emby-textarea::-moz-focus-inner {
border: 0;
}
.emby-textarea::-moz-focus-inner {
border: 0;
}

.textareaLabel {
display: inline-block;
Expand All @@ -44,15 +44,15 @@
padding-top: 0.5em;
}

.sso-role-mapping-container + .sso-role-mapping-container,
.sso-bordered-list + .sso-bordered-list {
margin-top: 1em;
}
.sso-role-mapping-container + .sso-role-mapping-container,
.sso-bordered-list + .sso-bordered-list {
margin-top: 1em;
}

.sso-role-mapping-container .sso-folder-list {
padding-bottom: 0.25em;
padding-left: 1em;
}
.sso-role-mapping-container .sso-folder-list {
padding-bottom: 0.25em;
padding-left: 1em;
}

.sso-role-mapping-input-label {
padding-left: 0.5em;
Expand Down Expand Up @@ -106,16 +106,16 @@ code {
height: 20px;
}

.eye-icon::before {
content: '';
position: absolute;
background-color: currentColor;
border-radius: 50%;
width: 8px;
height: 8px;
left: 5px;
top: 5px;
}
.eye-icon::before {
content: '';
position: absolute;
background-color: currentColor;
border-radius: 50%;
width: 8px;
height: 8px;
left: 5px;
top: 5px;
}

/* Slash to represent "hide" */
.eye-slash {
Expand All @@ -131,20 +131,20 @@ code {
color: #000; /* Eye icon hover color */
}

/* Tooltip */
.toggle-eye:hover::after {
content: "Show / Hide Token";
visibility: visible;
position: absolute;
width: 120px;
background-color: black;
color: white;
text-align: center;
border-radius: 5px;
padding: 5px;
bottom: 110%;
left: 50%;
transform: translateX(-90%);
font-size: 12px;
z-index: 999999;
}
/* Tooltip */
.toggle-eye:hover::after {
content: "Show / Hide Token";
visibility: visible;
position: absolute;
width: 120px;
background-color: black;
color: white;
text-align: center;
border-radius: 5px;
padding: 5px;
bottom: 110%;
left: 50%;
transform: translateX(-90%);
font-size: 12px;
z-index: 999999;
}
Loading

0 comments on commit 27acaad

Please sign in to comment.