Szabolcs Hubai

Results 157 comments of Szabolcs Hubai

Hi there, I am just a watcher of this repo, but want comment to this pull request! All of my test was on Win64: FF4.0.1 (empty profile) and Nightly 6.0a1...

I "patched" (=modify webnav to docshell) the omni.jar in my FF4.0.1 After patch it doesn't call my pseudo setCurrentURI(). :) And of course your fork gets working! With Nightly I...

hmmm. imho the omni.jar-hack isn't required (but recommend) for FF4: if you implement the "passthrou" method as I [said](https://gist.github.com/943273) then it will work (for FF4). :) of course, the compatibility...

ehh! my newly developed **omni.jar-hack v2** super-duper method brings your pull request to life under nightly! :) step1: get blunet/BarTab@1565e6a step2: "patch" `omni.jar/nsSessionStore.js` to `docShell` step3: delete `omni.jar/jsloader/resource/gre/components/nsSessionStore.js.bin` from archive...

After the Nighlty (6.0a1) -> Aurora (6.0a2) merge things gets bugging. :( For example with `Mozilla/5.0 (Windows NT 5.1; rv:6.0a2) Gecko/20110602 Firefox/6.0a2 ID:20110602042006` I getting `Error: browser.webNavigation._pauseLoadURI is not a...

I digged myself into it. Extending `BarTabWebProgressListener.onStateChange()` with [this gist](https://gist.github.com/1044942) will do the trick.

> `Error: browser.webNavigation._pauseLoadURI is not a function > Source File: resource://bartab/prototypes.js > Line: 693` IMO the landing of [Panorama Groups On Demand](https://wiki.mozilla.org/Firefox/Features/Panorama_Groups_On_Demand) caused that.

Opening a tab from `History / Recently Closed Tabs` menu or with `accel + shift + T` shortcut causes another `_pauseLoadURI()` error: `Error: browser.webNavigation._pauseLoadURI is not a function`

* [ `Base.checkApiKey`](https://github.com/openseedbox/openseedbox-server/blob/8c244153a3e6af7695ea5235ad9c73617b699352/app/controllers/Base.java#L13-L14)'s `@Before` should adjusted with [`unless`](https://www.playframework.com/documentation/1.3.x/controllers#before) * [routes](https://github.com/openseedbox/openseedbox-server/blob/8c244153a3e6af7695ea5235ad9c73617b699352/conf/routes#L11-L12) needs to be adjusted e.g. `/download/{token}/{hash}` See [answer](https://stackoverflow.com/a/26898656) for the question: [How can I generate temporary URL in play framework?](https://stackoverflow.com/questions/26898448/how-can-i-generate-temporary-url-in-play-framework)

For tokens: * [RandomWebToken.java](https://gist.github.com/lmammino/259a6b8e17e14a35d157dfc153e8d91f): ``` import java.security.SecureRandom; import java.util.Base64; import java.util.Base64.Encoder; public class RandomWebToken { public static void main(String[] args) { SecureRandom random = new SecureRandom(); byte bytes[] = new...