Skip to content

Version 1.1.2: java.lang.IllegalArgumentException: minPort (-1) and maxPort (-1) should be integers between 1024 and 65535. #113

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.

Sign up for GitHub

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

Jump to bottom
Closed
naive924 opened this issue Dec 1, 2020 · 4 comments
Closed

Version 1.1.2: java.lang.IllegalArgumentException: minPort (-1) and maxPort (-1) should be integers between 1024 and 65535. #113

naive924 opened this issue Dec 1, 2020 · 4 comments

Comments

@naive924
Copy link
Contributor

naive924 commented Dec 1, 2020

Version 1.1.2, with the default settings, will definitely fail to join the web conference with the following error.
It seems that it can be avoided with settings IP Address Mapping > Explicitly not use this mapping. > Disabled, and I will report it just in case.

  • Env
    local Windows environment

  • Stack trace

java.lang.IllegalArgumentException: minPort (-1) and maxPort (-1) should be integers between 1024 and 65535.
    at org.ice4j.ice.harvest.HostCandidateHarvester.checkPorts(HostCandidateHarvester.java:717)
    at org.ice4j.ice.harvest.HostCandidateHarvester.createDatagramSocket(HostCandidateHarvester.java:635)
    at org.ice4j.ice.harvest.HostCandidateHarvester.harvest(HostCandidateHarvester.java:352)
    at org.ice4j.ice.Agent.gatherCandidates(Agent.java:627)
    at org.ice4j.ice.Agent.createComponent(Agent.java:553)
    at org.jitsi.videobridge.transport.ice.IceTransport.<init>(IceTransport.kt:114)
    at org.jitsi.videobridge.transport.ice.IceTransport.<init>(IceTransport.kt:54)
    at org.jitsi.videobridge.transport.ice.IceTransport.<init>(IceTransport.kt)
@deleolajide
Copy link
Member

I will report it just in case.

image

Thanks for reporting. It is currently disabled. JVB2 does not appear to expose any parameters for this feature

@naive924
Copy link
Contributor Author

naive924 commented Dec 1, 2020

IP Address Mapping> Explicitly not use this mapping is the image here.

image

@deleolajide
Copy link
Member

image

I have experienced this issue already. That is why I disabled the port range for dynamic TCP ports when org.ice4j.ice.harvest.USE_DYNAMIC_HOST_HARVESTER is enabled. It is not supported in JVB2

IP address mapping is about binding JVB to specific network interfaces and/or IP address like this

image

This issue will be resolved in version 1.1.3

@naive924
Copy link
Contributor Author

naive924 commented Dec 2, 2020 •
edited
Loading

This issue will be resolved in version 1.1.3

Thank you! I will verify it again.

@gjaekel gjaekel mentioned this issue Dec 11, 2020
Closed
gjaekel added a commit to gjaekel/openfire-pade-plugin that referenced this issue Jan 26, 2021
@gjaekel @deleolajide
@1ay1 @cool0707 @okuhn @ClaudeStabile
pull upstream master 20202126 (#2)
f8255f4 
* add label to sip-communicator.properties file

* Fix NPE issue

* version 1.1.2

* Use the username instead of room nicks for participants in summary

* Update logging.properties

avoid double-timestamping

Fixes igniterealtime#97 for JiCoFo.

* Update logging.properties

avoid double-timestamping

* do not use jitsi jvm options for windows

* fix igniterealtime#101 - Bug or misconfiguration of colibri-ws Proxy?

* fix igniterealtime#96 - IP Address Mapping no longer takes when changed from Automatic to Disabled

* test for all windows platforms

* stop message notification flood

* fix igniterealtime#103 - Openfire meeting watermark images does not show up

* fix igniterealtime#102 - Look n Feel via android mobile Mic does not shown up

* bump pade version

* fix igniterealtime#105 - Add support for configuring the new video quality settings

* fix igniterealtime#104 - Enhancement: Cam don't switch offline if Video is disabled

* update changelog

* Update JvbPluginWrapper.java

Along with domain, make port of public URL of colibri-ws announced to the *client* configurable.

* add public port for jvb websockets

* make jvb2 websockets mandatory

* refresh with latest jicofo build

* refresh with latest jvb build

* Update JitsiJicofoWrapper.java

Pass inverted Property `ofmeet.conference.auto-moderator` as config property  `org.jitsi.jicofo.DISABLE_AUTO_OWNER` inside the Jicofo config file.

The tag name `ofmeet.conference.auto-moderator` is chosen with respect to `ofmeet.conference.admin`

* added back ofgasi

* added new settings to jigasi for sip proxy

* added support for jigasi

* bump version

* remove test config file

* work-in-progress

* add support for jigasi plugin and fix issues

* update version to 1.1.3

* prepare for 1.1.3

* tidy up websocket code

* provide dummy file

* remove focus and callcontrol components

* build and test on linux

* Update ConfigServlet.java

Refactor a possible bug: In case of _stereo_ enable, the config tag `stereo` was putted twice to config.

* ensure max bitrate for screenshare

* Update ConfigServlet.java

Make _audioLevels_ an configurable option.
Most people advice to have disabled this, therefore introduce the inverted tag `ofmeet.audioLevels.enabled` with default `false`.

* Update ofmeet_i18n.properties

Clarify (at least to me!) misleading description.

* Update ofmeet-settings.jsp

refactor label name

* Update ofmeet_i18n.properties

refactor label

* Update ofmeet-settings.jsp

refactor 2nd tag name

* Update ofmeet_i18n.properties

refactor 2nd tag name

* Update custom_ofmeet.js

Make audio toggling as short as possible to keep privacy;  un-muting of 1s will dispose recognizable contents to the other participants.

I already suggest this with my first contribution, but you (Dele) changed it for some reason unknown to me.

* Update custom_ofmeet.js

Enhance resolution of default "initials" avatar by 4 to 128px.

For unknown reasons, with OFMeet V1.1 this code path is finally used! And my work on an appropriate order of initials come into place.

* Update JitsiJicofoWrapper.java

fixed type clash

* Update custom_ofmeet.js

* Update custom_ofmeet.js

* Update custom_ofmeet.js

* Update custom_ofmeet.js

* Update confetti.browser.min.js

Here's the Xmas Edition of the confetti tool!

I did a bit un-minify to allow better editing. In this first sprint, the shapes set for Xmas is hard coded. On may refactor to externalize this later.

The code change is taken from the upstream pull request catdad/canvas-confetti#111 , which is not processed for now. Because it's not restricted to emojis but may be used with any text, I changed the token used for declaration.

* remove redundant files

* video quality work-in-progress

* Update confetti.browser.min.js

Move Xmas shape set to `custom_ofmeet.js`

* Update custom_ofmeet.js

Change shape set for the confetti tool to an *Xmas edition* during December.

ToDo: Add an easter egg.

* Update custom_ofmeet.js

more snow flakes to keep more balance with other shapes

* Update custom_ofmeet.js

* empty classes/jitsi-meet folder

* copy modified files to correct location

* revert back to stable release 5142 for jvb  and jicofo

* add more video quality settings to admin web page

* add visual indicators for jvm, jicofo and jigasi services

* version 1.1.3

* change default background to igniterealtime fans image

* fix igniterealtime#75
fix igniterealtime#110
fix igniterealtime#114
fix igniterealtime#113
fix igniterealtime#106
fix igniterealtime#82

* remove testing data

* Update custom_ofmeet.js

* make websocket data channel optional and configurable

* merge jigasi and jicofo into ofmeet plugin

* fix igniterealtime#132 fix igniterealtime#119 fix igniterealtime#210

* Update InterfaceConfigServlet.java

Make `DISABLE_VIDEO_BACKGROUND` configurable by the System Property `org.jitsi.videobridge.ofmeet.disable.video_background`

Disabling the video background will remove flickering on fine-lined screen shares.

* Update README.md

* Update README.md

* Update README.md

* Add FreeSWITCH integration to Jigasi

* make jigasi and freeswitch services enable configurable

* fix igniterealtime#139

* remove quotes. linux does not like them

* fix igniterealtime#139 fix igniterealtime#137 fix igniterealtime#111

* Update InterfaceConfigServlet.java

Enable enforced notification auto-dismiss with a timeout of 15s.

Solves igniterealtime#142.

* fix igniterealtime#145 - Enhancement: Deliver Administrative Messages to Meeting Participants

* fix igniterealtime#144 - Enhancement: add several options to meeting settings

* prepare for version 1.2.0

* fix igniterealtime#149 - Regression: Remove "stay on page" on leaving a conference

* improve detection of active conference recording

* version 1.2.0

* fix igniterealtime#150 - Cosmetical: Quirk in Status Table on Summary Page

* add FS JSPs from deprecated FS plugin

* Fix character encoding problem on welcome page.

* Update JitsiJigasiWrapper.java

* Update README.md

* Update README.md

* Update README.md

* version 1.2.1

* Update README.md

* Update README.md

* Update README.md

* Update WellKnownHostMeta.java

Allow to override the annouced port of the XMPP server, `httpbind.port.secure` (with the default `7443`), by a public view  defined by `httpbind.publicport.secure`.

* fix igniterealtime#120 - Dominant Speaker Switching don't work

* version 1.2.2

* Update ofmeet_i18n.properties

Enhance description for Data Channel via Web Socket

Supports igniterealtime#120

* final ver 1.2.2

* Update README.md

Please also alter the Image included at L47-

* Update README.md

* Update README.md

* Update README.md

* further validation on colibri-ws port. prevent 8080 and 8888

* Update README.md

for the simplest case, one may use iptables to redirect a port with just one rule
```
iptables -A PREROUTING -i eth0 -p tcp -m tcp --dport 443 -j REDIRECT --to-ports 7443
```

* Update custom_ofmeet.js

Fix broken avatar image in case the information taken from the vCard PHOTO is incomplete. This may happen if the vCard is pulled by LDAP at an WinAD.

May fix igniterealtime#168.

* fix igniterealtime#154 - Start Order of Plugins OFMeet and Pade

* blank jvm default not working with linux

* use correct plugin name

* fix igniterealtime#176 - Meeting planner does not show up in Meeting plugin

* prepare for release 1.3.0

* cosmetic changes

* fix igniterealtime#178 - Remove webinar mode

* add accents to pade name

* added ofmeet changelog

* Update custom_ofmeet.js

Sort names in Contact Manager.

Support igniterealtime#50.

* fix igniterealtime#50 - Enhancement: Sort Entries in Contact Manager

* Update custom_ofmeet.js

Make the icon area of a contact clickable.

Fixes igniterealtime#51.

* Update custom_ofmeet.js

Use UI message type _error_, which will stay until dismissed.

Enhances igniterealtime#145.

* cosmetic changes

* update changelog

* use versioning with pade client maven build

* fix syntax error

* Update JitsiJvbWrapper.java

Note: The (internal) default binding for the JVB private interface is localhost. Use `local_ip`, if another `public_ip` is in use, because then it should be save to allow access from the local net.

Also drop former used `keystore`

* Update JitsiJvbWrapper.java

In local mode, bind to all interfaces.

* Update JitsiJvbWrapper.java

Remove support to configure the address of the private REST API for security reasons. Without this, it will default to `localhost`.

* cosmetic change

* Update JitsiJvbWrapper.java

Fix a missing use of `rest_port` instead of former fixed `8080`.

* fix igniterealtime#182 - Feature Request: Allow access to JVB REST-API via private http interface

* fix igniterealtime#163 - Support VP9 codec

* disable bandwidth estimation (bwe) when vp9 forced

* Add list of meetings in progress.

* Fix i18n Pade corruption

* Fix Comment COOL0707 -> BAO

* fix igniterealtime#186 - Feature Request: List of meetings in progress on Welcome page

* fix igniterealtime#186 - Don't add non public rooms to meetings in progress list

* fix igniterealtime#186 - use forwarded url if available

* Update InProgressListServlet.java

Use refer(r)er URL to reflect external (client browser) view of the URL used for meetings.

* Create ofmeet-de.json

German translation for the "Meetings in Progress" feature.

* Update custom_ofmeet.css

To keep display clean, make "green" (good) connection state indicator hidden after a while

* revert igniterealtime#190

* fix igniterealtime#193 - Custom branding.js file to be preserved

* remove webinar code again

* fix igniterealtime#189 - Enhancement: Clean-up display by hiding green "good connection" state inidcator

* cosmetic changes

* Update README.md

* Update custom_ofmeet.css

Allow the Conferences Display at the Welcome Page to show more entires.

* fix igniterealtime#193 - Custom branding.js file to be preserved

* fix igniterealtime#197 - Enhancement: Enlarge Conferences Display

* Update custom_ofmeet.css

Make the right Toolbar wrapping around in the same way as the left one.

* add ofmeetUrl to branding.js

* added missing stop method on websocket client object

* fix NPE in BookmarkInterceptor

* add fastpath webchat to url redirect exclusion list

* update to jicofo 2.0.5390

* update to jvb 2.0.5390

* disable sctp for windows

* bump version to 1.3.1

* Avoild creation of unused HttpClients

At the backend connection of the WebSocket reverse proxy, there's no need to have a pool of HttpClients as provided by default.

* Create SslContextFactoryProvider.java

Provide a SslContextFactory

* Create QueuedThreadPoolProvider.java

Provide a common ThreadPool used by the HttpClient

* update changelog

* Update JitsiJvbWrapper.java

* enable REST-endpoints for health check (`/about/health`) and debug (`/debug`)
* brush-up indents and rearrange configuration sections

* fix igniterealtime#213 - Enhancement: Enable some useful REST-endpoints of JVB2

* Localize the view of custom_ofmeet

* fix #14 - cyrillic with some variables

* Update custom_ofmeet.js

Add a event handler to auto-hide the mouse pointer after an idle timeout of 10s.

Support igniterealtime#218 .

* fix igniterealtime#210 - APP.conference.addConferenceListener has been removed

* upgrade jitsi-meet

* fix igniterealtime#218 - Feature: Auto-hide mouse pointer

* Fixed initial letter avatars to be centered correctly.

* testing git

* fix igniterealtime#226 - Fixed initial letter avatars to be centred correctly

* igniterealtime#225 new feedback servlet

* igniterealtime#225 read feedback form from file

* igniterealtime#225 feedback post request

* igniterealtime#225 fixed wrong post location

* igniterealtime#225 writing post data to log

* igniterealtime#225 log raw post data

* igniterealtime#225 log parts

* igniterealtime#225 adding @MultipartConfig to servlet

* igniterealtime#225 read value from part inputstream

* Update FeedbackServlet.java

* brush code and format style
* Introduce macro replacing

* Update feedback.html

introduce macros

* Update pom.xml

Add Apache POI

* Update feedback.html

Introduce separate ratings for audio and video

* Update ConfigServlet.java

Introduce `ofMeetConfig.getFeedbackEnabled()` to add `enableClosePage`

* Update ConfigServlet.java

Use `ofmeet.feedback.enabled` to controll output of `enableClosePage`

* Update InterfaceConfigServlet.java

Introduce `ofmeet.feedback.enable` to control output of `SHOW_PROMOTIONAL_CLOSE_PAGE`

* fix build error

* Update FeedbackServlet.java

Add missing ìmport`

* work-in-progress

* Update FeedbackServlet.java

* Get macros from property repository 
* Brush-Up code syle

* updated feedback form

* fix i18n issue with feedback defaults

* Update branding.js

Added settings variables 👍   
  "publishLocation"         : {disable: false, value: true},
    "enableHomePage"          : {"disable": true, "value": false},
    "homePage"                : {"disable": true, "value": false},
    "ofmeetUrl"               : {"disable": false, "value": "https://pade.chat:5443/ofmeet/"},

* Update feedback.html

re-introduce separate ratings for audio and video.
Use "star" icons for now and a headline

* Update FeedbackServlet.java

* use `optString` to get JSON properties because they might be missing.
* update auditlog message, add a marker if it contain a comment message.

* Update branding.js

added     "showToolbarIcons"        : {"disable": false, "value": true},
and corrected some typo in file

* fix igniterealtime#225

Co-authored-by: Dele Olajide <[email protected]>
Co-authored-by: 0xBAAAAAAD <[email protected]>
Co-authored-by: るしふぁ <[email protected]>
Co-authored-by: Gitea <[email protected]>
Co-authored-by: Oliver Kuhn <[email protected]>
Co-authored-by: Claude Stabile <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@deleolajide @naive924