klutter icon indicating copy to clipboard operation
klutter copied to clipboard

Silently fails to add server

Open elliotrpage opened this issue 2 years ago • 11 comments

I recently switched phones from a pixel 4a to 7a. On the new phone I am unable to add my Komga server and each time I try it silently fails.

I am using known good credentials and have attempted using both the local IP/port and the DNS. No matter what permutation of information I put in, it silently fails and no server is added.to the list.

I am running Klutter 0.6 on android 13. I had a look in the app to see if there are logs I can provide but cannot find them.

Any ideas?

elliotrpage avatar May 30 '23 17:05 elliotrpage

Same issue here on a Galaxy S23.

NeoDuder avatar Jun 05 '23 09:06 NeoDuder

Same problem on my new Pixel tablet.

bcurran3 avatar Jun 22 '23 20:06 bcurran3

Same here too unfortunately. Samsung tab

ellipsiss avatar Jul 04 '23 06:07 ellipsiss

The cause of this has been identified as Klutter using now deprecated API paths (v1).

This will require a new release to correct the paths to the new version, as well as testing to make sure the new API paths behave with Klutter.

On Tue, 4 Jul 2023 at 07:10, ellipsiss @.***> wrote:

Same here too unfortunately. Samsung tab

— Reply to this email directly, view it on GitHub https://github.com/frameset/klutter/issues/15#issuecomment-1619558662, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEIV7NKWPNIZJSFYT6QPP3XOOXWJANCNFSM6AAAAAAYUJY7LY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

frameset avatar Jul 04 '23 08:07 frameset

Thanks for the update!

ellipsiss avatar Jul 04 '23 09:07 ellipsiss

Thank you. But this may take a while. If Klutter is a must for you, consider using the last version before v1 of Komga.

On Tue, 4 Jul 2023 at 10:09, ellipsiss @.***> wrote:

Thanks for the update!

— Reply to this email directly, view it on GitHub https://github.com/frameset/klutter/issues/15#issuecomment-1619854500, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEIV7IEVZRJSH25WGOJUN3XOPMU5ANCNFSM6AAAAAAYUJY7LY . You are receiving this because you commented.Message ID: @.***>

frameset avatar Jul 04 '23 14:07 frameset

@frameset Thank you for your hard work!

nicolasshu avatar Jul 20 '23 06:07 nicolasshu

@frameset, thanks for your work.

Do you know when the new release supporting v1.0.0 will be out?

Thanks

PS: i think you can only change v1 to v2 on your https://github.com/frameset/klutter/blob/master/lib/data/dataproviders/client/user_controller.dart file.

i will give a test if i found some time.

nonameboy007 avatar Jul 24 '23 11:07 nonameboy007

hi,

i was able to build an apk with latest package:

dany@dany-G55VW:~/klutter$ cat pubspec.yaml name: klutter description: A client for streaming comics from a Komga comic/manga/BD server.

The following line prevents the package from being accidentally published to

pub.dev using pub publish. This is preferred for private packages.

publish_to: 'none' # Remove this line if you wish to publish to pub.dev

The following defines the version and build number for your application.

A version number is three numbers separated by dots, like 1.2.43

followed by an optional build number separated by a +.

Both the version and the builder number may be overridden in flutter

build by specifying --build-name and --build-number, respectively.

In Android, build-name is used as versionName while build-number used as versionCode.

Read more about Android versioning at https://developer.android.com/studio/publish/versioning

In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.

Read more about iOS versioning at

https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html

version: 0.7.0+14

environment: sdk: ">=2.12.0 <3.0.0"

dependencies: flutter: sdk: flutter built_value: ^8.0.0 cached_network_image: ^3.1.0 dio: ^5.3.0 dio_cache_interceptor: ^3.0.0 dio_cache_interceptor_hive_store: ^3.0.1 equatable: ^2.0.3 flutter_bloc: ^8.1.3 flutter_launcher_icons: ^0.13.1 flutter_secure_storage: ^8.0.0 fullscreen: ^1.0.3 hydrated_bloc: ^9.1.2 intl: ^0.18.1 json_annotation: ^4.0.1 path_provider: ^2.0.2 photo_view: ^0.14.0 pretty_dio_logger: ^1.1.1 retrofit: ^4.0.1 rotated_corner_decoration: ^2.1.0+1 rxdart: ^0.27.1 sizer: ^2.0.13 url_launcher: ^6.0.6 uuid: ^3.0.4 validators: ^3.0.0

dev_dependencies: flutter_test: sdk: flutter build_runner: ^2.0.4 built_value_generator: ^8.0.0 json_serializable: ^6.7.1 retrofit_generator: ^7.0.8

flutter_icons: image_path: "assets/images/icon.png" image_path_ios: "assets/images/icon_ios.png" android: true ios: true remove_alpha_ios: true

flutter: assets: - assets/images/

uses-material-design: true

on emulator, the login is working with komga > 1.0.

But on my s22 ultra, the package doesnt want to install, i will check later, need to sleep now.

Cheers

nonameboy007 avatar Jul 25 '23 01:07 nonameboy007

@frameset, thanks for your work.

Do you know when the new release supporting v1.0.0 will be out?

Thanks

PS: i think you can only change v1 to v2 on your https://github.com/frameset/klutter/blob/master/lib/data/dataproviders/client/user_controller.dart file.

i will give a test if i found some time.

We need to make sure that the objects returned by the new API version can be consumed by the data models in Klutter, otherwise there could be some kind of error. It isn't just as simple as flicking the version number up in the API controller url strings, though I admit that may just work for many cases.

frameset avatar Jul 27 '23 16:07 frameset

Yes I have some error on series, and books cant be read...

nonameboy007 avatar Jul 27 '23 16:07 nonameboy007