Merge pull request #1067 from cryzed/fix/server-preserve-focus

Fix `Fix Preserve Focus` option not working on Linux servers

README.md

@@ -86,7 +86,7 @@ specific packages.
 
 **Q: Does drag and drop work on linux?**
 
-> A: No *(see [#855](https://github.com/debauchee/barrier/issues/544) if you'd like to change that)*
+> A: No *(see [#855](https://github.com/debauchee/barrier/issues/855) if you'd like to change that)*
 
 
 **Q: What OSes are supported?**

doc/newsfragments/linux-drag-drop-faq.doc

@@ -0,0 +1 @@
+Fixed FAQ link to Linux drag and drop issue.

doc/newsfragments/restore-dpiawareness.bugfix

@@ -0,0 +1 @@
+Fixed a regression in 2.4.0 that caused Barrier to not support scaling other than 100% (https://github.com/debauchee/barrier/issues/1462).

src/cmd/barrierc/CMakeLists.txt

@@ -21,7 +21,9 @@ set(sources
 if (WIN32)
     file(GLOB arch_headers "MSWindows*.h")
     file(GLOB arch_sources "MSWindows*.cpp")
-    list(APPEND sources barrierc.rc)
+    list(APPEND sources
+        barrierc.rc
+        barrierc.exe.manifest)
 elseif (APPLE)
     file(GLOB arch_headers "OSX*.h")
     file(GLOB arch_sources "OSX*.cpp")

src/cmd/barrierc/barrierc.exe.manifest

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
+    <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
+        <security>
+            <requestedPrivileges>
+                <requestedExecutionLevel level="asInvoker" uiAccess="false"></requestedExecutionLevel>
+            </requestedPrivileges>
+        </security>
+    </trustInfo>
+    <application xmlns="urn:schemas-microsoft-com:asm.v3">
+        <windowsSettings>
+            <dpiAware xmlns="http://schemas.microsoft.com/SMI/2005/WindowsSettings">true</dpiAware>
+            <dpiAwareness xmlns="http://schemas.microsoft.com/SMI/2016/WindowsSettings">PerMonitor</dpiAwareness>
+        </windowsSettings>
+    </application>
+</assembly>

src/cmd/barriers/CMakeLists.txt

@@ -21,7 +21,9 @@ set(sources
 if (WIN32)
     file(GLOB arch_headers "MSWindows*.h")
     file(GLOB arch_sources "MSWindows*.cpp")
-    list(APPEND sources barriers.rc)
+    list(APPEND sources
+        barriers.rc
+        barriers.exe.manifest)
 elseif (APPLE)
     file(GLOB arch_headers "OSX*.h")
     file(GLOB arch_sources "OSX*.cpp")

src/cmd/barriers/barriers.exe.manifest

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
+    <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
+        <security>
+            <requestedPrivileges>
+                <requestedExecutionLevel level="asInvoker" uiAccess="false"></requestedExecutionLevel>
+            </requestedPrivileges>
+        </security>
+    </trustInfo>
+    <application xmlns="urn:schemas-microsoft-com:asm.v3">
+        <windowsSettings>
+            <dpiAware xmlns="http://schemas.microsoft.com/SMI/2005/WindowsSettings">true</dpiAware>
+            <dpiAwareness xmlns="http://schemas.microsoft.com/SMI/2016/WindowsSettings">PerMonitor</dpiAwareness>
+        </windowsSettings>
+    </application>
+</assembly>

src/gui/res/mac/Info.plist

@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple Computer//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
   <dict>

src/lib/arch/unix/ArchDaemonUnix.cpp

@@ -20,7 +20,6 @@
 
 #include "arch/unix/XArchUnix.h"
 #include "base/Log.h"
-#include "barrier/App.h"
 
 #include <unistd.h>
 #include <sys/types.h>
@@ -87,8 +86,6 @@ ArchDaemonUnix::daemonize(const char* name, DaemonFunc func)
         break;
 
     default:
-		// first restore terminal settings
-		App::instance().getEvents()->restoreTerminal();
         // parent exits
         exit(0);
     }

src/lib/barrier/FileChunk.cpp

@@ -98,7 +98,7 @@ FileChunk::assemble(barrier::IStream* stream, String& dataReceived, size_t& expe
     case kDataChunk:
         dataReceived.append(content);
         if (CLOG->getFilter() >= kDEBUG2) {
-                LOG((CLOG_DEBUG2 "recv file chunck size=%i", content.size()));
+                LOG((CLOG_DEBUG2 "recv file chunk size=%i", content.size()));
                 double interval = stopwatch.getTime();
                 receivedDataSize += content.size();
                 LOG((CLOG_DEBUG2 "recv file interval=%f s", interval));

src/lib/base/EventQueue.h

@@ -63,7 +63,6 @@ public:
     virtual Event::Type getRegisteredType(const std::string& name) const;
     void*                getSystemTarget();
     virtual void        waitForReady() const;
-	virtual void        restoreTerminal() const { m_parentStream.restoreTerminal(); }
 
 private:
     UInt32                saveEvent(const Event& event);

src/lib/base/IEventQueue.h

@@ -184,11 +184,6 @@ public:
     be added.
     */
     virtual void        waitForReady() const = 0;
-    //! Restore the terminal settings
-    /*!
-    Called to restore the terminal settings after daemonizing.
-    */
-	virtual void        restoreTerminal() const = 0;
 
     //@}
     //! @name accessors

src/lib/base/NonBlockingStream.cpp

@@ -57,10 +57,4 @@ bool NonBlockingStream::try_read_char(char &ch) const
     return false;
 }
 
-void NonBlockingStream::restoreTerminal() const
-{
-    tcsetattr(_fd, TCSANOW, _p_ta_previous);
-    fcntl(_fd, F_SETFL, _cntl_previous);
-}
-
 #endif // !defined(_WIN32)

src/lib/base/NonBlockingStream.h

@@ -39,7 +39,6 @@ public:
     ~NonBlockingStream();
 
     bool try_read_char(char &ch) const;
-	void restoreTerminal(void) const;
 
 private:
     int _fd;

src/lib/net/SecureUtils.cpp

@@ -190,7 +190,7 @@ void generate_pem_self_signed_cert(const std::string& path)
 
     X509_sign(cert, private_key, EVP_sha256());
 
-    auto fp = fopen_utf8_path(path.c_str(), "r");
+    auto fp = fopen_utf8_path(path.c_str(), "w");
     if (!fp) {
         throw std::runtime_error("Could not open certificate output path");
     }

src/lib/platform/XWindowsScreen.cpp

@@ -324,7 +324,7 @@ XWindowsScreen::leave()
     m_impl->XGetInputFocus(m_display, &m_lastFocus, &m_lastFocusRevert);
 
 	// take focus
-	if (m_isPrimary || !m_preserveFocus) {
+	if (!m_preserveFocus) {
         m_impl->XSetInputFocus(m_display, m_window, RevertToPointerRoot, CurrentTime);
 	}
 

src/test/mock/barrier/MockEventQueue.h

@@ -64,5 +64,4 @@ public:
     MOCK_METHOD0(forClipboard, ClipboardEvents&());
     MOCK_METHOD0(forFile, FileEvents&());
     MOCK_CONST_METHOD0(waitForReady, void());
-    MOCK_CONST_METHOD0(restoreTerminal, void());
 };