Hi,

It is now working.  It seemed to be a combination of problems:

1. There was a problem with the proxy I was using which has now been fixed.

2. When the proxy contained "http://"; it did not work.  Instead I changed it
back to just "my.proxy.com" and set QNetworkProxy type to HttpProxy.

My final code for the create method looked like this:

QNetworkProxy proxy;
proxy.setHostName("my.proxy.com");
proxy.setPort(3128);
proxy.setType(QNetworkProxy::HttpProxy);


I still have one unanswered question from before :)

I removed a call to the static method

QNetworkProxy::setApplicationProxy(proxy);

... because of the need for create(QObject* parent) to be re-entrant.
Is it OK to call this static method from a re-entrant method?  If so, should
the documentation make this clear?

Thank you for your help too :)

On Thu, Apr 29, 2010 at 7:01 AM, Martin Jones <martin.jo...@nokia.com>wrote:

> On Wed, 28 Apr 2010 11:12:11 pm ext Jack Wootton wrote:
> > To follow up on this.  I have been reading the documentation for
> > QNetworkProxyFactory:
> > http://doc.qt.nokia.com/4.7-snapshot/qnetworkproxyfactory.html
> >
> > For the method
> >
> > QList<QNetworkProxy> QNetworkProxyFactory::systemProxyForQuery ( const
> > QNetworkProxyQuery & query = QNetworkProxyQuery() )   [static]
> >
> > it is documented that: "On Windows, this function will use the WinHTTP
> DLL
> > functions. Despite its name, Microsoft suggests using it for all
> > applications that require network connections, not just HTTP. This will
> > respect the proxy settings set on the registry with the proxycfg.exe
> tool.
> > If those settings are not found, this function will attempt to obtain
> > Internet Explorer's settings and use them."
> >
> > I'm have set my proxy using proxycfg tool on Windows, however I still get
> > no proxy results from QNetworkProxyFactory::systemProxyForQuery
>
> QNetworkProxyFactory::systemProxyForQuery() requires a QNetworkProxyQuery
> to
> be passed to it.  This would also require using a QNetworkProxyFactory,
> rather
> than simply setting a proxy for the QNetworkAccessManager.  If you want to
> do
> this then you can see the qml runtime source for inspiration -
> tools/qml/qmlruntime.cpp (search for SystemProxyFactory).
>
> To begin, I think you should get the code below working, though.  I suspect
> the reason it does not work is that you have not provided a valid host
> name:
>
>     proxy.setHostName("my.proxy.com<http://my.proxy.com>");
>
> changing it to this should make it work:
>
>     proxy.setHostName("my.proxy.com");
>
> assuming my.proxy.com is a valid proxy.
>
> I've added a simple example to examples/declarative/proxyviewer,
> illustrating
> how to use a QDeclarativeNetworkAccessManagerFactory (should appear on
> gitorious within 24hrs).  The code is essentially the same as you have
> below,
> so with a correct proxy set this should work.
>
> Martin.
>
> > On Tue, Apr 27, 2010 at 2:35 PM, Jack Wootton
> > <jackwoot...@gmail.com<mailto:jackwoot...@gmail.com>> wrote: Hello,
> >
> > I have modifed the my implementation of
> > QDeclarativeNetworkAccessManagerFactory (question follows code) :)
> >
> > MyNetworkAccessManagerFactory.h
> > ===========================
> > class MyNetworkAccessManagerFactory : public
> > QDeclarativeNetworkAccessManagerFactory {
> > public:
> >      MyNetworkAccessManagerFactory();
> >      ~MyNetworkAccessManagerFactory();
> >
> > public:
> >     QNetworkAccessManager* create ( QObject * parent );
> > };
> >
> >
> > MyNetworkAccessManagerFactory.cpp
> > =============================
> > MyNetworkAccessManagerFactory::MyNetworkAccessManagerFactory()
> > {
> > }
> >
> > MyNetworkAccessManagerFactory::~MyNetworkAccessManagerFactory()
> >
> > {
> > }
> >
> > QNetworkAccessManager* MyNetworkAccessManagerFactory::create ( QObject*
> > parent ) {
> >      QNetworkProxy proxy;
> >      proxy.setHostName("my.proxy.com<http://my.proxy.com>");
> >
> >      proxy.setPort(3128);
> >
> >      // Is this static method call acceptable for a re-entrant method?
> >      // QNetworkProxy::setApplicationProxy(proxy);
> >
> >      QNetworkAccessManager* nam = new QNetworkAccessManager(parent);
> >
> >
> >      nam->setProxy(proxy);
> >      return nam;
> > }
> >
> >
> >
> > I have a question about ensuring the method create(QObject* parent) is
> > thread safe / re-entrant: I assumed the method call
> > 'QNetworkProxy::setApplicationProxy(proxy);' is not ok since it is static
> > and should not be called from a re-entrant function.  Is this the case?
> > If so, is it a problem that it is not called?
> >
> > I made the changes, but I still don't see the webpage loaded in the QML
> > document, but I'll deal with that after I know my code is correct first
> :)
> >
> >
> >
> > On Tue, Apr 27, 2010 at 2:19 AM, Martin Jones
> > <martin.jo...@nokia.com<mailto:martin.jo...@nokia.com>> wrote:
> >
> > On Tue, 27 Apr 2010 01:18:56 am ext Jack Wootton wrote:
> > > Hi,
> > >
> > > I'm trying to set some proxy settings for a QML application - it's not
> > > working as expected.  I'm unsure the API is being used correctly, can
> > > someone check my code makes sense? There are 3 files: main.cpp,
> > > MyNetworkAccessManagerFactory.h and MyNetworkAccessManagerFactory.cpp
> > >
> > > I'm using qt 4.7 and followed the documentation
> > > (
> http://doc.qt.nokia.com/4.7-snapshot/qdeclarativenetworkaccessmanagerfac
> > > t ory.html) when writing the code.  The method
> > > MyNetworkAccessManagerFactory::create(QObject* parent) does get called
> > > when the QML document is loaded using the QDeclarativeView, so the
> > > framework is definitely picking up the new proxy settings.
> > >
> > > The questions I have are:
> > >
> > > 1. Is the implementation of MyNetworkAccessManagerFactory::create
> > > correct? I assumed because I was subclassing from a class with
> 'factory'
> > > in its name, that it was ok to return the same pointer to
> > > NetworkAccessManager.
> > >
> > > 2. How do I know if the method that calls "create" takes ownership of
> the
> > > returned pointer or not?  I assumed it didn't, hence cleaning up in
> > > ~MyNetworkAccessManagerFactory();
> > >
> > > 3. Is there anything obviously wrong with the way I'm using the
> > > NetworkAccessManager?
> >
> > The factory must create a  new NAM each time it is called.  This is
> because
> > QML creates multiple threads and each must have its own NAM.  It is
> > important that you set the parent of the NAM created to the parent passed
> > in the create() method (which you already do).  The NAM will be destroyed
> > when its parent is destroyed.  I will update the documentation to make
> > this clear.
> >
> > Also take special note of this line from the documentation:
> >
> > "Note: the create() method may be called by multiple threads, so ensure
> the
> > implementation of this method is reentrant."
> >
> > --
> > Martin
> >
> >
> >
> > --
> > Regards
> > Jack
> >
> >
> >
> > --
> > Regards
> > Jack
>
> --
> Martin
>



-- 
Regards
Jack
_______________________________________________
Qt-qml mailing list
Qt-qml@trolltech.com
http://lists.trolltech.com/mailman/listinfo/qt-qml

Reply via email to