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

v0.1.13-alpha Bug Reports: Add your bugs here! #58

Closed
E3V3A opened this issue May 3, 2014 · 2 comments
Closed

v0.1.13-alpha Bug Reports: Add your bugs here! #58

E3V3A opened this issue May 3, 2014 · 2 comments
Labels

Comments

@E3V3A
Copy link
Contributor

E3V3A commented May 3, 2014

NOTE:
@xLaMbChOpSx There is something weird about the notifications I get on github about your latest release. It points to un invalid url:
https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/notifications
https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/releases/tag/v0.1.132-alpha

The side swipe is fantastic!

Latest findings:

  • In Preferences: the "persistence" check logic is still reversed.
  • Menu: "Update OpenCellID Data", now gives the error message
    "Unable to determine your last location! Enable location services (GPS) and try again."
  • Menu: add "About" with version information so that users verify they run the correct version.
  • When disabling "tracking cell info", it should also disable "location tracking". But not vice versa.

Unclear: Not sure how this is done, but obviously it should not turn off GPS if people are using it for something else, but it should turn off the AIMSICD data collection of positions. Actually, when thinking more about this, perhaps we should just move this location tracking into settings or just remove it all-together, since the cell tracking data (without) GPS data to compare with is little worth? So perhaps the location tracking could/should be done automatically and transparently to user? We could still show the icon, to inform the user that location tracking is on, but that is kind of redundant since there will already be both a GPS and a AIMSICD icon in top bar.

@E3V3A
Copy link
Contributor Author

E3V3A commented May 3, 2014

@xLaMbChOpSx
Added:

  • In the "aimsicd-cellinfo.csv", there's still an entry with CellID set to "-1". I think this is just a timing issue, so perhaps you could add a short timer before collecting data and in addition (?) check that it is not "-1"...

Improvement:

  • In the Map viewer, can you change the cell tower text from "CellID - nnnn" to "CellID: nnnn". The minus/dash sign throw me off every time..

@Nothing4You
Copy link

@E3V3A
The first URL you listed as broken works fine for me; the second one has an extra 2 in it, not sure where you got that from, it should be https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/releases/tag/v0.1.13-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants