Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extractor: added Rabbitstream #536

Merged
merged 3 commits into from
Aug 5, 2023
Merged

Conversation

Sofie99
Copy link
Contributor

@Sofie99 Sofie99 commented Aug 4, 2023

fix sflix bundle aswell

Copy link
Contributor

@LagradOst LagradOst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool 👍
Just a minor code quality and link need changing

callback: (ExtractorLink) -> Unit
) {
val id = url.substringAfterLast("/").substringBefore("?")
val rawKey = app.get(key).text
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be cached with smth like

private var rawKey : String? = null 
suspend fun getRawKey() : String = rawKey ?: app.get(key).text.also { rawKey = it }

private fun decryptSourceUrl(decryptionKey: ByteArray, sourceUrl: String): String {
val cipherData = base64DecodeArray(sourceUrl)
val encrypted = cipherData.copyOfRange(16, cipherData.size)
val aesCBC = Cipher.getInstance("AES/CBC/PKCS5Padding")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add ?: throw ErrorLoadingExeption("Cipher not found")

so you dont need to do Objects.requireNonNull and !! later

override val mainUrl = "https://dokicloud.one"
}

open class Rabbitstream : ExtractorApi() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like you are using something related to https://github.com/enimax-anime/enimax please add some sort of link to that in the top of the file along with the licence of enimax

@Sofie99 Sofie99 requested a review from LagradOst August 4, 2023 15:48
@Sofie99
Copy link
Contributor Author

Sofie99 commented Aug 4, 2023

Ready to merge

@LagradOst LagradOst merged commit bbbb7c4 into recloudstream:master Aug 5, 2023
2 checks passed
@Sofie99 Sofie99 deleted the Rabbitstream branch August 25, 2023 08:41
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.

None yet

2 participants