BUGFIX: http->https in skyview

[keflavich]

Resolves Issue #3345, i.e., the upstream bug in which queries hang indefinitely.

If you've arrived here from another issue, please install the latest astroquery version. The skyview problems reported in June 2025 have been solved.

• TODO: add test (currently, I have a failing test)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.08%. Comparing base (1becbba) to head (0ecf641).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3346   +/-   ##
=======================================
  Coverage   70.08%   70.08%           
=======================================
  Files         232      232           
  Lines       19890    19890           
=======================================
  Hits        13940    13940           
  Misses       5950     5950           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[bsipocz]

Hahh, I guess I better push small fixes more frequently, I have that doctest fixed on a local branch since April 😅

[bsipocz]

(haven't notice it in time, but please use the magic words right before the PR number next time

[keflavich]

Ahhh crap, I can't say "resolves issue X", I have to say "resolves X"!?!? argh, I'm a terrible magician

[carlos-lco]

Hi there. Thanks a lot for tip Adam.
In fact, I piped to the new version using "--pre", changing from 0.4.10 to 0.4.11,
and now the SkyView works.
However, Astroplan uses the function SkyView with an argument (grid)
that seems deprecated in the new version. So astroplan users probably will need
to request an update on the "finder" function as well.

[keflavich]

@carlos-lco thanks for the note; that seems like an issue that needs to be posted on astroplan - is there one open?

[carlos-lco]

Yes, SkyView # 588. I just noticed it. And you wrote on it! I should investigate more before posting. Sorry.

[keflavich]

No worries. That was a long time ago (in my head anyway), I forgot about it. Would you be interested in trying to fix that issue? I'm sure a pull request would be welcome.

[bsipocz]

This latest change in skyview is really just about the server side turning off the http access due to their DoS attacks. , the underlying API has not changed. (That being said, it would be great if downstream would fix these sooner rather than later).

[carlos-lco]

No worries. That was a long time ago (in my head anyway), I forgot about it. Would you be interested in trying to fix that issue? I'm sure a pull request would be welcome.

I just did. Crossed fingers. :)