Feature_JS_Support_Library #49
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ibution tar ball.
…nc_p which requires proxy code changes accordingly.
. sslfido.cpp, wsfido.cpp, wsfido.h: made the output message more clear to distinguish between different handshakes.
. sslfido.cpp, wsfido.cpp, wsfido.h: made the output message more clear to distinguish between different handshakes.
…-frmwrk into Feature-JS-Support-Library
. rpcfg.py: fixed a bug 'Config WebServer' button does not work if the 'WS installer' checkbox is not checked.
…copy when adding a websocket frame header in the front of the data payload.
… and CIncomingPacket.
. enums.js: added more property ids
…ion bug introduced when adding config cache to save repeated parsing of json files, which kept only one copy of interface configurations, and will be modified when authentication is enabled, and caused the CRpcTcpBridgeAuth object created afterwards having all its interfaces enabled before login succeeds, which shall not be allowed. The solution is to make a copy of the matches from the cach for each CRpcTcpBridgeAuth instance before making modifications to the matches.
…d by the newly added RebuildMatches to CRpcTcpBridgeAuth and CRpcTcpBridgeProxyAuth
…sed, and caused infinite loop.
'ForwardRequest'. . configdb.js, enums.js, proxy.js, dmsg.js: made some minor changes.
…g was not serialized in the correct format. . rpcroute.cpp: fixed a memory leak caused when adding duplicated remote-match. It was not a problem till the EnableRemoteEvent was changed from CIfTransactGroup to CIfTaskGroup, and the ChangeRelation did not work to stop further actions when the match to add was found already added. So the solution is to change back to CIfTransactGroup to get ChangeRelation work properly. . defines.js, index.js, enablevt.js, fwrdreq.js, iomgr.js, proxy.js, bridge.js: added synchronization for the requests of 'EnableRemoteEvent' to complete before the 'Start' process ends, so that we can move on to send user requests. and added code to stop the CInterfaceProxy gracefully. And continued adding code to ForwardRequestLocal
…e.js, dmsg.js: fixed some bugs to get request 'ForwardRequest' to work.
…string in serialization/deserialization. index.js, proxy.js, bridge.js: made modifications to make sure the startup promises are completed in the expected sequence when handshake is failed.
… index.js: added handler for the request 'ForwardEvent'.
. index.js, actcancel.js, asynctst.js, evtest.js: refacted index.js and splitted to three test cases, actcancel, asynctst and evtest. . enablevt.js, fwrdevt.js, iomgr.js, proxy.js: removed DispatchEventMsg from proxy.js and directed the ForwardRevent event to ForwardEventLocal. . bridge.js, fwrdreqrmt.js: fixed exception when ForwardRequest failed after connection is lost.
. index.js, actcancel.js, evtest.js, asynctst.js: moved creation and start of g_oIoMgr from the testcases to the index.js. . actcancel.js, asynctst.js: fixed a bug the oContext was mistakenly shared between and caused actcancel to cancel the task from asynctst, and failed.
duplicated websockets to the same destURL.
…r side socket, so that if the server binds to 0.0.0.0, the propDestIpAddr gives the true ip address instead of this wildcard address. KeepAlivesvr.h, katest/maincli.cpp: updated the outdated code with the latest ridlc.
. enums.js: fixed a typo that propCallFlags and propCallOptions shared the same value. . enablevt.js, fwrdreq.js, iomsg.js, bridge.js: fixed a bug in CIoReqMessage that the time counter m_dwTimeLeftSec was initialized or reset using the second as the unit while the timer decrease this value with delta of milliseconds, and caused the keepalive request to timeout quickly. . fwrdreq.js, fwrdreqrmt.js: added support of one-way request . fwrdevtrmt.js: fixed wrong handling of strRouterPath . bridge.js: fixed a bug in OpenPortQueue, that stopped handling the pending openport requests when found the port was already created. . iomgr.js: fixed a bug, the m_mapProxies maps the portid to the proxy instances, which is wrong since multiple proxy instances may share the same portid, and the fix made a change to map the portid to a set of proxy instances.
|
I have finished half of the support library. Next I will move on to implement the stream related features. |
console.log with DebugPrint from CInterfaceProxy, for better output.
commit. The iomgr's m_mapProxies was changed to be a map between portid and a set of proxies, and the proxy's Stop was not updated and sent the CloseRemotePort request directly to the webworker, which is not correct and should let the iomgr to determine if all the proxies to the same port id are released before it sends out the CloseRemotePort request to the webworker. . actcancel.js: made a change to Stop the websocket after the request is canceled.
…cancel.js: made some changes to CInterfaceProxy's Start and Stop, to make the start/stop process more reliable.
testcase with the latest ridlc. . reqopen.cpp, bridge.cpp, : removed duplicated code. . tcpfido.cpp, bridge.cpp: fixed crashes if the client sends ill-formed OpenStream request to the rpcrouter. . tcportex.cpp: replaced map erasing by key with erasing by iterator.
…s: added handlers for 'FetchData' and 'OpenStream' requests. . iomsg.js, cancelrq.js, enablevt.js, fwrdreq.js, keepalive.js, enablevtrmt.js, actcancel.js, katest.js, asynctst.js: replaced 'propTimeoutsec' with 'propTimeoutSec' . iomgr.js, fwrdreqrmt.js: fixed some bugs
…enums.js, iomsg.js, iomgr.js, openstm.js, proxy.js, bridge.js, stmtest.js: added all the stream related functions. . configdb.js: added code to restore bytearray property to Buffer . fwrdevt.js: added support for serialization protocol 'seriNone' . openstmrmt.js: fixed a bug the fetchdata request's response was not genenrated correctly because of using 'undefined' req message instead of the right one.
…mwrite.js, bridge.js, enablevtrmt.js, fwrdreqrmt.js, handshake.js, openstmrmt.js, stream.js, stmtest.js: fixed some bugs to get stream's basic funtions to work
. stream.js: added error handling on test 'CanSend' failed. . asynctst.js: eliminated the hanging promise. . stmtest.js, webpack.config.js: make some minor adjustment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Continue the development of JS support library interrupted by issue #46.